Skip to content

Conversation

@jesstelford
Copy link
Contributor

This has significant speed improvements on large codebases by reducing the files scanned to just those with changes.

Copy link
Member

@sam-b-rose sam-b-rose left a comment

Choose a reason for hiding this comment

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

Nice! Love this workflow optimization ⚡ ty!

@sam-b-rose sam-b-rose added the 🤖Skip Changelog Causes CI to ignore changelog update check. label Oct 28, 2022
@alex-page alex-page merged commit f14d986 into main Oct 28, 2022
@alex-page alex-page deleted the fast-commit-commands branch October 28, 2022 14:53
# Bring back the change with "polaris-migrator:" comments
git stash pop

# (optional) if there a files that have both "polaris-migrator:" comments
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# (optional) if there a files that have both "polaris-migrator:" comments
# (optional) if there are files that have both "polaris-migrator:" comments

git stash pop

# (optional) if there a files that have both "polaris-migrator:" comments
# _and_ complete fixes, add the complete fixes now
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# _and_ complete fixes, add the complete fixes now
# and complete fixes, add the complete fixes now

@alex-page
Copy link
Member

alex-page commented Oct 28, 2022

@samrose3 sorry I hit the merge button as it seemed logical/small to me. Feel free to follow up with these docs updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖Skip Changelog Causes CI to ignore changelog update check.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants