Skip to content

added git workflow update_git_index_after_ignoring_files.yaml#80

Merged
alokedesai merged 3 commits intowarpdotdev:mainfrom
vjindal0112:main
May 12, 2022
Merged

added git workflow update_git_index_after_ignoring_files.yaml#80
alokedesai merged 3 commits intowarpdotdev:mainfrom
vjindal0112:main

Conversation

@vjindal0112
Copy link
Copy Markdown
Contributor

Description of changes (updated or new workflows)

added git workflow update_git_index_after_ignoring_files.yaml

It covers the case of regenerating the git index after ignoring files which were previously committed to your repository.

Copy link
Copy Markdown
Member

@alokedesai alokedesai left a comment

Choose a reason for hiding this comment

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

Thank you so much for contributing--this looks great! Just have one comment about the tags

Comment on lines +11 to +12
- gitignore
- version control
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

in our experience, the best workflows have fewer tags (see https://github.com/warpdotdev/workflows#what-makes-a-useful-workflow). Given that--could we remove the gitignore and version control flags?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done!

@vjindal0112 vjindal0112 requested a review from alokedesai May 12, 2022 17:48
Copy link
Copy Markdown
Member

@alokedesai alokedesai left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

Thanks so much @vjindal0112! Looking forward to seeing you soon :)

@alokedesai alokedesai merged commit 34e0a8b into warpdotdev:main May 12, 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

Successfully merging this pull request may close these issues.

2 participants