Skip to content
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

[ELY-2829] Replace wildcard imports with specific imports inside OidcSecurityRealmTest.java #2224

Closed
wants to merge 8 commits into from

Conversation

Samith10
Copy link
Contributor

@Samith10 Samith10 commented Oct 4, 2024

@luck3y
Copy link
Contributor

luck3y commented Oct 4, 2024

@Samith10 It looks like you included some extra commits in this one. I think perhaps you're just missing the step that when you've completed a PR you can do:
$ git checkout 2.x
then make a new branch for your next issue:
$ git checkout -b ELY-2829
and this won't include any prior changes and you can proceed with fixing the issue, committing and pushing the changes.

@SoniaZaldana
Copy link
Contributor

To add to what @luck3y said, you can then take your last commit from this branch and apply it to your new branch as follows:
git cherry-pick <commit-hash>

Seeing how this is the relevant commit, you would do
git cherry-pick e1d02f153d7d96b1c6f61c216b86c7df7606c9c9

That way you don't have to redo everything :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants