8000 Different minor changes by creyD · Pull Request #123 · creyD/prettier_action · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Different minor changes #123

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 20 commits into from
May 8, 2025
Merged

Different minor changes #123

merged 20 commits into from
May 8, 2025

Conversation

creyD
Copy link
Owner
@creyD creyD commented Jul 1, 2023 •
edited
Loading
  • Added option no_commit
  • Default working directory is now the github.action_path
  • Set prettier_version default to latest
  • Added option only_changed_pr
  • Added option git_identity

@creyD creyD self-assigned this Jul 1, 2023
@chicco785 @creyD
Support not exiting the process in case of changes in dry mode (#122)
4bd40fc 
Co-authored-by: Conrad <grosserconrad@gmail.com>
@creyD creyD added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Jul 20, 2023
@chicco785 chicco785 mentioned this pull request Nov 17, 2023
Closed
creyD and others added 11 commits January 10, 2024 12:26
@creyD
Updated Readme according to #132
117a7a5
@creyD
Fixing bug #128
f32e6a9
@jetmore
set INPUT_WORKING_DIRECTORY if not already set
a3561a8
@creyD
Merge pull request #133 from greenfiling/fix-empty-working_directory
9991486
@creyD
added codacy badge
d5e6c38
@jetmore
Add only_changed_pr (#135)
e53f331 
* allow `only_changed` and `dry` to work together

Before this change, if you tried to use `dry` and `only_changed together, it acted as if `only_changed` was ignored.  There was no code specific to `only_changed` down the `dry` code path.

This commit’s solution is to run prettier on the entire code base (as was already being done) and then, before doing any comparisons, reset changes to files that were not modified in the last commit.

* add `only_changed_pr` option

This behaves similarly to only_changed, except instead of only considering files which were modified in the last commit, this option only considers files which were modified in any commit included in the current PR.
@creyD
fix: codacy warning
12efc36
@creyD
fix: fixed codacy error
eeb940d
@creyD
fix: minor documentation cleanups
9f54668
@creyD
fix: version bump for examples
f24ab29
@creyD
fix: updated link
b73c2bf
@creyD creyD merged commit d7f059e into master May 8, 2025
2 checks passed
@david-allison david-allison mentioned this pull request May 9, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@creyD creyD

Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@creyD @chicco785 @blakedunson @DerekTBrown @jetmore
0