Skip to content

GitHub Reference feature is in wrong package #1019 #1040

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

Jayanta-karmakar
Copy link
Contributor

@Jayanta-karmakar Jayanta-karmakar commented Mar 2, 2024

fixes #1019
The GitHub reference feature is located in the commands.github but should be in features.github instead like any other feature. so, I rebased commands.github to features.github, and made some methods public so that they can be accessible.

@Jayanta-karmakar Jayanta-karmakar requested review from a team as code owners March 2, 2024 07:52
@CLAassistant
Copy link

CLAassistant commented Mar 2, 2024

CLA assistant check
All committers have signed the CLA.

Taz03
Taz03 previously approved these changes Mar 2, 2024
@christolis
Copy link
Member

Great job @Jayanta-karmakar! Only thing that's left to do now is to get the Spotless checks to pass. Was the pre-commit hook executed when creating the commit? You should use the command ./gradlew spotlessApply in order to make them pass!

christolis
christolis previously approved these changes Mar 2, 2024
Copy link
Member

@christolis christolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Taz03
Taz03 previously approved these changes Mar 2, 2024
@christolis christolis linked an issue Mar 2, 2024 that may be closed by this pull request
Copy link
Member

@SquidXTV SquidXTV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the GitHubCommand.java file still in commands\github? The full feature is supposed to be in features\github

@Jayanta-karmakar
Copy link
Contributor Author

Okay @SquidXTV, I will Fix it

@Jayanta-karmakar Jayanta-karmakar dismissed stale reviews from Taz03 and christolis via 4d79184 March 5, 2024 18:52
@Jayanta-karmakar
Copy link
Contributor Author

I have checked everything and checked all the code and ran all the tests... it looks good to me

@Taz03 Taz03 self-assigned this Mar 20, 2024
Taz03
Taz03 previously approved these changes Mar 20, 2024
ankitsmt211
ankitsmt211 previously approved these changes Mar 20, 2024
Copy link
Member

@ankitsmt211 ankitsmt211 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 👍
Thank you 😊

@Taz03 Taz03 merged commit 24391fe into Together-Java:develop Mar 20, 2024
@ankitsmt211 ankitsmt211 mentioned this pull request Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub Reference feature wrong package
6 participants