Skip to content

Add sorter for projects.json #100

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

Closed
wants to merge 1 commit into from
Closed

Add sorter for projects.json #100

wants to merge 1 commit into from

Conversation

iranzo
Copy link
Contributor

@iranzo iranzo commented Mar 23, 2018

This code sorts projects.json alphabetically, so after this, a new commit is needed on projects.json

@castastrophe
Copy link
Contributor

Are there any dependencies to run this script? Does the user need to have python installed? I'm wondering because I want to make sure our package.json contains all dependencies for working inside the repo.

Could this be written using bash instead so we could add it as an npm task? npm run sort?

@castastrophe castastrophe self-requested a review March 24, 2018 13:41
@castastrophe castastrophe self-assigned this Mar 24, 2018
@castastrophe castastrophe added the question Further information is requested label Mar 24, 2018
@iranzo
Copy link
Contributor Author

iranzo commented Mar 24, 2018

Hi, should just have json dependency, in bash it would require other approach.

Still could be executed from time to time on project before new web push

@castastrophe
Copy link
Contributor

I'd love to incorporate this task into the build workflow and I think the best way to do that would be to write this as part of the build script. I love this idea though and am only closing this PR out in favor of using a different tool.

@ananya26-vishnoi ananya26-vishnoi mentioned this pull request May 29, 2021
14 tasks
@vsoch vsoch mentioned this pull request Jun 2, 2022
14 tasks
@razo7 razo7 mentioned this pull request Sep 14, 2022
14 tasks
This was referenced Jan 19, 2023
@Yasir761 Yasir761 mentioned this pull request Jan 20, 2023
14 tasks
@jasonmadigan jasonmadigan mentioned this pull request May 4, 2023
14 tasks
@avdhendra avdhendra mentioned this pull request Jul 27, 2023
14 tasks
@yatmagazin yatmagazin mentioned this pull request May 25, 2024
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants