Skip to content

Conversation

huats
Copy link
Contributor

@huats huats commented Jan 24, 2025

No description provided.

huats added 4 commits January 24, 2025 11:54
Signed-off-by: Christophe Sauthier <[email protected]>
Signed-off-by: Christophe Sauthier <[email protected]>
Signed-off-by: Christophe Sauthier <[email protected]>
Signed-off-by: Christophe Sauthier <[email protected]>
**/*.svg
**/*.png
**/*.json
**/token
Copy link
Contributor

@RobertKielty RobertKielty Jan 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@huats I was looking at this list of file types working thru it to consider if they had any impact on the external GHA work and I was thinking that it would probably be fine.

But then I noticed that there is a json file in this PR (suggestions.json) are you sure you want to **/*.json files in .gitgnore moving forward?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your feedback. Happy to keep the .json out of the gitignore file if needed of course. But since it is a working directory that has (on my end) a lot of tokens and credentials it was simply easier like this...
Please let me know what you prefer and I'll follow your judgement

@jeefy jeefy force-pushed the main branch 16 times, most recently from 4199bf0 to b926aab Compare May 6, 2025 03:47
@jeefy jeefy merged commit c533db2 into main May 11, 2025
4 of 7 checks passed
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.

3 participants