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

Fix check_style.sh refactor #10862

Merged
merged 2 commits into from
Jan 28, 2021
Merged

Conversation

cameel
Copy link
Member

@cameel cameel commented Jan 28, 2021

Fixes bugs introduced in #10797. Looks like I did not test it thoroughly enough.

Note: It's still a draft because I included a commit with intentional style error to make sure the script works this time. I'll remove it and mark as ready to review once I see that comments are getting posted correctly. It's ready now.

@cameel cameel self-assigned this Jan 28, 2021
@cameel cameel force-pushed the fix-wrong-script-path-in-check-style-sh branch 2 times, most recently from db9a2e5 to 1afe24c Compare January 28, 2021 11:21
@cameel cameel changed the title Fix wrong script path in check_style.sh Fix check_style.sh refactor Jan 28, 2021
- The script is actually in a subdirectory.
@cameel cameel force-pushed the fix-wrong-script-path-in-check-style-sh branch from cc6e422 to 922134d Compare January 28, 2021 11:37
@cameel cameel force-pushed the fix-wrong-script-path-in-check-style-sh branch from 922134d to cd4b667 Compare January 28, 2021 11:41
@cameel cameel marked this pull request as ready for review January 28, 2021 11:44
@cameel
Copy link
Member Author

cameel commented Jan 28, 2021

It's now ready for review and the chk_coding_style job is passing but let's make sure all jobs pass before merging just in case.

@cameel cameel merged commit d753c1a into develop Jan 28, 2021
@cameel cameel deleted the fix-wrong-script-path-in-check-style-sh branch January 28, 2021 12:58
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