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

Ditch csharpier-repos #128

Closed
belav opened this issue Apr 20, 2021 · 2 comments
Closed

Ditch csharpier-repos #128

belav opened this issue Apr 20, 2021 · 2 comments

Comments

@belav
Copy link
Owner

belav commented Apr 20, 2021

Instead of having a mono repo of public files that is essentially a snapshot of the code from them, we can hopefully do the following.

  1. Add a json file with a list public repos to check out
  2. Add a .csharpierrc file that includes the files that will be ignored from the public repos
  3. Modify the github action for format_repositories to check out all repositories from the json file into the same location as the .csharpierrc, then format them all.

This means we can easily add new repos and we continue to test the latest versions of those repos, which should help catch more edge cases.

@jrobertson-insite
Copy link

  1. Should be .csharpierignore see comment on issue 140

@belav
Copy link
Owner Author

belav commented Apr 26, 2021

@belav belav modified the milestones: 0.9.1, Beta, 0.9.2 May 2, 2021
@belav belav modified the milestones: 0.9.3, 0.9.4 May 16, 2021
@belav belav modified the milestones: 0.9.4, 0.9.5 May 24, 2021
@belav belav closed this as completed Feb 5, 2022
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

No branches or pull requests

2 participants