Skip to content

Conversation

@dulmandakh
Copy link

@dulmandakh dulmandakh commented Sep 10, 2021

This PR makes it possible to exclude files from format checking. For example, https://github.com/dulmandakh/buck/runs/3570342055 in need to exclude .idea and some other files.

Also removes node_modules, and compiles to single dist/index.js (optimized) file, see https://github.com/actions/javascript-action.

Fixes #14

@dulmandakh dulmandakh changed the title allow to exclude files from formatting exclude files from formatting Sep 10, 2021
@dulmandakh dulmandakh changed the title exclude files from formatting exclude files from format Sep 10, 2021
@axel-op
Copy link
Owner

axel-op commented Sep 11, 2021

Hello @dulmandakh,

Thanks for this very useful pull request! Please ignore the failing checks. Before I merge it, could you please separate the changes related to file selection and those related to the dependencies in two different commits?

@axel-op axel-op force-pushed the master branch 2 times, most recently from e95a8d6 to 230e739 Compare March 12, 2022 20:49
axel-op and others added 4 commits March 12, 2022 22:27
Co-authored-by: Tariq Yusuf <[email protected]>
Co-authored-by: DanielF <[email protected]>
Co-authored-by: Tariq Yusuf <[email protected]>
Co-authored-by: DanielF <[email protected]>
Co-authored-by: Tariq Yusuf <[email protected]>
Co-authored-by: DanielF <[email protected]>
@axel-op
Copy link
Owner

axel-op commented Mar 12, 2022

@danielferromeral I've merged your commits from https://github.com/dulmandakh/googlejavaformat-action/pull/1 into @dulmandakh's branch and edited them. I've renamed the exclude argument to files-excluded.

I will now merge this PR. Thanks to both of you 😊

@axel-op axel-op merged commit 1cc75be into axel-op:master Mar 12, 2022
@axel-op
Copy link
Owner

axel-op commented Mar 12, 2022

The changes have been published under the new v3.6.0 release. You can pinpoint this version in your workflow:

      - uses: axel-op/[email protected]

The changes have also been integrated into the v3 branch, which you can reference instead:

      - uses: axel-op/googlejavaformat-action@v3

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.

Skip formatting specific Java files

3 participants