Commit Graph

9 Commits

Author SHA1 Message Date
Jordan Reimer
3684e898c5 removes ui_copywrite from pre-commit hook (#24448) 2023-12-08 10:45:30 -07:00
Hamid Ghaf
db1170576f only update license changes in ui related files in ui precommit hook (#24313) 2023-11-30 22:47:38 +00:00
Jordan Reimer
a0562419ce Copywrite pre-commit (#24091)
* adds pre-commit function for adding copyright headers for files in ui directory

* removes additional log level change

* fixes issue looping staged files

* adds comment

* updates to stage changes after adding headers rather than blocking commit

* cleanup comments and unused code from ui_copywrite bash function
2023-11-14 15:32:17 -07:00
Mike Palmiotto
a9778be3f2 Only run fmtcheck on changed files pre-commit (#21747)
The pre-commit hook was taking quite a while to run. Let's just check
changed files to avoid unnecessary `go fmt` on the entire tree.
2023-07-11 10:26:16 -04:00
Alexander Scheel
b0fb3b1420 pre-commit hook for formatting backend code (#21682)
* Add backend format linting to pre-commit hook

By taking a slight penalty with each commit, we can ensure that
contributors follow the format behavior by default (if they run hooks),
making accidental PRs without proper formatting less likely.
Additionally, fix gofmtcheck to align with the Makefile, fixing the
corresponding fmtcheck target for use with the hook.

Signed-off-by: Alexander Scheel <alex.scheel@hashicorp.com>

* Fix formatting errors

Signed-off-by: Alexander Scheel <alex.scheel@hashicorp.com>

---------

Signed-off-by: Alexander Scheel <alex.scheel@hashicorp.com>
2023-07-10 16:49:47 +00:00
Marc Boudreau
1361932d91 Remove CircleCI Configuration (#19751)
* modify pre-commit hook to not look for circleci

* remove circleci configuration
2023-03-24 14:24:06 -04:00
Marc Boudreau
58bbcad74b Marcboudreau/vault 13760/add test UI to completed (#19747)
* remove check of circleci configuration from pre-commit hook

* add dependency on test-ui for tests-completed job
2023-03-24 13:20:49 -04:00
Sam Salisbury
b39176b923 pre-commit: no fail if circleci missing or too old (#6990)
Just a warning instead.
2019-06-26 15:49:42 +01:00
Sam Salisbury
380f460eff ci: break config into separate files (#6849)
* ci: break config into separate files

* Untangle githooks

* githooks: fix whitespace

* .hooks/pre-commit: add ui -> lint-staged check

- We no longer require dependency on husky with this change.

* ui: remove husky dependency and config

- The previous commit obviates the need for it.
- We will now have to manage these hooks by hand, but this removes
  the conflict between husky-installed hooks and those in the .hooks dir.

* ui: update yarn.lock with husky removed

* .hooks/pre-commit: always use subshell + docs

- Always use subshell means we consistently exit from the
  same place which feels less complex.
- Docs are necessary for horrible bash like this I think...

* Makefile: remove old husky githooks

- Husky has installed a handler for every single git hook.
- This causes warnings on every git operation.
- Eventually we can remove this, but better not to confuse
  people with these messages for now.

* ci: fix go build tags

* Makefile: improve compatibility of rm call

- Looks like the xargs in Travis does something different to the one
  on my mac, this more verbose call should be safe everywhere.

* ci: fix make target names

* ci: fix test-ui invocation

* Makefile: simplify husky hook cleanup

* ci: more focussed readme
2019-06-11 09:55:53 -05:00