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

Add check for tabs vs spaces to GitHub Actions for Pull Requests #2011

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

kylemhall
Copy link
Member

This change adds another test to the GitHub actions that trigger when a pull request is made. This test will check to see if any lines have been indented using spaces instead of tabs.

Test Plan:

  1. Include this in a branch
  2. Make a pull request for that branch
  3. Introduce another commit that uses spaces instead of tabs 3) Note the test fails!
  4. Introduce another commit that changes those spaces to tabs 5) Push to your remote pull request branch
  5. Note the tests pass!

This change adds another test to the GitHub actions that trigger
when a pull request is made. This test will check to see if any
lines have been indented using spaces instead of tabs.

Test Plan:
1) Include this in a branch
2) Make a pull request for that branch
3) Introduce another commit that uses spaces instead of tabs
3) Note the test fails!
4) Introduce another commit that changes those spaces to tabs
5) Push to your remote pull request branch
6) Note the tests pass!
@kylemhall
Copy link
Member Author

@kylemhall
Copy link
Member Author

@mdnoble73 mdnoble73 merged commit 0f31f56 into Aspen-Discovery:24.09.00 Aug 22, 2024
2 checks passed
@kylemhall kylemhall deleted the qa_check_tabs branch August 30, 2024 10:46
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.

2 participants