Skip to content
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.

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

Format Ramble's object files with black #521

Merged
merged 3 commits into from
Jun 5, 2024

Conversation

linsword13
Copy link
Collaborator

@linsword13 linsword13 commented Jun 3, 2024

  • Add support for formatting object files (apps and mods) in the style cmd
  • Add a new config file for these object files, to accommodate for the different line-length requirement they have compared with primary files
  • Run ramble style -af to auto-format these object files

Also a (small but separate) change:

  • Do not fallback to checking all-files when no file is changed (this can happen say on a new checkout of develop, or a branch that only modifies files that are not in the style check scope.)

* Add in a separate config file for the object files (apps and mods,) to
  accommodate for the different line-length requirements between primary files and
  object files
* Update style cmd to apply black separately for primary and object
  files
@douglasjacobsen douglasjacobsen merged commit cc7a83e into GoogleCloudPlatform:develop Jun 5, 2024
11 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.

2 participants