-
Notifications
You must be signed in to change notification settings - Fork 7
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
Ensure Table of Contents is sorted alphabetically #10
Comments
Kindly Check in I kinda tweaked with this somehow |
Checking it out @Chibuike-edmund ! Appreciate the Pull Request and implementing it! |
Adding Pull Request #11 Contributed by @Chibuike-edmund |
Tried the test merge, see the Actions log: https://github.com/raymelon/scripting-recipes/actions/runs/6495866893 Encountered this. Resolved by using updated versions of actions (v3) as suggested here Node.js 12 to 16 See resolution at Commit c0ca9 |
Hi @Chibuike-edmund, kindly take a look at the Job Log I tried adding an entry on the top of Bash section, but the GitHub Action did not update the README file. |
is your issue that the other types are not sorted? I think this is the case as the code only looks at python segments. if you want the others to be changed a simple refactor to the code can be done |
@georgettica Hi, thank you for your comment. You're right, the script only looks at python segments. With that, I also did the same thing for the Python segment. The Action ran, but the README file is not updated still. Is the step
|
It's more on each PR it will add a commit and push there than on the main branch. And it's possible |
Closing the issue. I modified update_readme to work as I intended. Appreciate everyone who contributed! |
An ideal solution is to implement an automated workflow that will update README.md > Table of Contents.
Workflow flow:
3.1. Extract script filenames
3.2. Sort the script filenames alphabetically
3.3. Rewrite the language section
Anyone interested to implement this, let me know. Will include this in Hacktoberfest 2023.
I'm also open to discussions, feel free to reply with your suggestions.
The text was updated successfully, but these errors were encountered: