Skip to content

Conversation

@gadenbuie
Copy link
Collaborator

@gadenbuie gadenbuie commented Feb 22, 2024

This PR re-organizes our make check target to perform the checks in the build workflow in GHA. It also creates make check-fix that fixes formatting issues instead of just checking them.

I also organized all of the check targets, e.g. check-format, check-types, check-tests, etc. leaving the existing well-known targets, e.g. format, pyright, test, etc.

The relevant public targets are now

check                check code, style, types, and test (basic CI)
check-fix            check and format code, style, types, and test
pyright              check types with pyright
lint                 check style with flake8
test                 check tests quickly with the default Python
format               format code with black and isort

Copy link
Collaborator

@schloerke schloerke 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!

Co-authored-by: Barret Schloerke <[email protected]>
@gadenbuie gadenbuie merged commit 71638d2 into main Feb 22, 2024
@gadenbuie gadenbuie deleted the chore/make-check branch February 22, 2024 17:08
schloerke added a commit that referenced this pull request Feb 27, 2024
* main:
  Update `@render.data_frame` renderer to work properly with fill/fillable (#1126)
  Don't error on malformed date input (#1139)
  fix(nav_panel): Add `.bslib-gap-spacing` if nav panel is fillable (#1154)
  feat: Add `ui.input_dark_mode()` (#1149)
  chore(make): Add `make docs` and `make docs-preview` (#1150)
  Sync with latest bslib sidebar and card changes (#1129)
  docs(input_task_button): Add to reference index, also update methods for action button/link (#1151)
  chore(make): Update check and check-fix to replicate CI (#1148)
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.

3 participants