Skip to content
This repository has been archived by the owner on Feb 29, 2024. It is now read-only.

Remove usage of codeowner-checker from project and pulled in required classes #11

Merged
merged 7 commits into from
Oct 16, 2023

Conversation

poloka
Copy link
Contributor

@poloka poloka commented Oct 12, 2023

Description of Changes

  • Remove references to codeowner-checker as project is read-only and pulled in dependencies

NOTE: Because the code pulled in under lib/codeowners was from another project, ignore the review of these and refactoring is not part of this code review but rather to just pull in the files as is for usage.

How Has This Been Tested?

specs

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Release activity

Checklist:

Following the Contributing Guidelines

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works and they pass when merging upstream
  • I have added an entry to the Changelog accordingly
  • I have added my name to the CONTRIBUTORS.md
  • I have squashed related commits

@poloka poloka changed the title Update supported ruby to 3.2 Remove usage of codeowner-checker from project and pulled in required classes Oct 12, 2023
jessica-bailey
jessica-bailey previously approved these changes Oct 12, 2023
@poloka poloka merged commit bb4290c into main Oct 16, 2023
@poloka poloka deleted the UpdateRuby branch October 16, 2023 18:56
@poloka poloka linked an issue Oct 16, 2023 that may be closed by this pull request
1 task
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update to support Rails 6
3 participants