Fix indentation error in scripts#38
Open
Nonoka-T wants to merge 3 commits into
Open
Conversation
Contributor
Author
|
@sehmaluva Hi, I've fixed indentation error. Please kindly check my PR. Thank you! |
Owner
|
@Nonoka-T This is good by the .idea directory is still tracked. You need to pull the latest changes from main to fix it |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📋 Description
This PR resolves a CI failure caused by pre-existing syntax issues in two script files.
What type of contribution is this?
What does this PR do?
This PR fixes IndentationError in two files that were causing the Multi-Language Testing / test-python CI workflow to fail on every pull request
The indentation of the affected lines has been corrected to match the surrounding code blocks, allowing flake8 to run without E999 errors.
Related Issue
🎯 For Beginners: Step-by-Step Guide
Before Submitting
✅ I have tested my changes locally
✅ I have followed the project guidelines
✅ I have updated documentation (if needed)
Need Help?
🧪 Testing
How to test this PR?
Test Results
📸 Screenshots / Demo
N/A
Before
After
📝 Checklist
Code Quality
Documentation
/docsTesting & Validation
Git & Commit
🎓 For Different Contribution Types
📱 Profile Contributions
Profile Checklist
docs/templates/PROFILE_TEMPLATE.mdprofiles/YOUR_GITHUB_USERNAME.mdProfile Tips
🧩 Challenge Solutions
Challenge Checklist
challenges/level/challenge-name/)Challenge Tips
🚀 Project Contributions
Project Checklist
Project Tips
📝 Documentation
Documentation Checklist
Documentation Tips
🌟 Additional Information
What I learned while working on this:
How a single misplaced indentation can cause flake8 to raise E999 SyntaxError which blocks the entire CI workflow before any other checks can run.
Challenges I faced:
The first flake8 run only revealed one error per file. After fixing it, a second indentation error appeared further down in the same file.
Questions for reviewers:
📚 Resources I Used
💬 Notes for Reviewers