Commit Graph

4 Commits

Author SHA1 Message Date
Scott Anderson 2b7b5f03c7 Reduce workflow permissions (#748)
Reduce workflow permissions to the minimum required.

Co-authored-by: Scott Anderson <662325+scottanderson@users.noreply.github.com>
2025-05-13 20:26:22 -07:00
Scott Anderson 95f5cf68d2 Cleanup deploy script (#708)
## Description:

- Remove the environment from the deploy action, to prevent github
actions from automatically creating a second deployment. This is no
longer needed because the relevant secrets have been moved up to the
repository level.
- Remove the PR comment, as this is now redundant.


![image](https://github.com/user-attachments/assets/e07fde04-9788-4a5e-9a94-01b0917c45d1)

## Please complete the following:

- [x] I have added screenshots for all UI updates
- [x] I confirm I have thoroughly tested these changes and take full
responsibility for any bugs introduced
- [x] I understand that submitting code with bugs that could have been
caught through manual testing blocks releases and new features for all
contributors

---------

Co-authored-by: Scott Anderson <662325+scottanderson@users.noreply.github.com>
2025-05-13 19:57:45 -04:00
Scott Anderson 5de469e312 Update workflow files (#675)
## Description:

- Split the CI workflow in to separate build and test jobs, to improve
visibility on PRs when the build passes but tests fail.
- Add emoji labels to various jobs and steps
- Add build and startup time to job summary


![image](https://github.com/user-attachments/assets/187e99d2-20de-4220-b1a4-11f4d2e72647)

![image](https://github.com/user-attachments/assets/83f9a9fc-c118-415a-92ca-faca48bac31e)

![image](https://github.com/user-attachments/assets/b8061a28-7d5d-4f93-99c6-673f3ca84e30)

## Please complete the following:

- [x] I have added screenshots for all UI updates
- [x] I confirm I have thoroughly tested these changes and take full
responsibility for any bugs introduced
- [x] I understand that submitting code with bugs that could have been
caught through manual testing blocks releases and new features for all
contributors

Co-authored-by: Scott Anderson <662325+scottanderson@users.noreply.github.com>
2025-05-07 19:15:48 -07:00
Scott Anderson 9a3983d014 ESLint improvements (#388)
## Description

Adds various features to improve the DX with eslint:

### Webpack

- Use `eslint-webpack-plugin` for better observability of lint errors
during development.

### Ignored files

- Enable `.gitignore` support for eslint.

### Commit hook

- Add `eslint --fix` to pre-commit hook.

### Github actions
- Add eslint check to a new github action workflow.
- Use `eslint-formatter-gha` to annotate PR files with lint failures
(see the Files changed tab,
[example](https://github.com/openfrontio/OpenFrontIO/pull/388/commits/73862230be5aad7b18e122b1cd4ab05fc9570b2c)).


![image](https://github.com/user-attachments/assets/b399b92e-8eb0-48e1-92b7-ce630ccf12e1)

## Testing

These changes have been validated through local testing and through
Github workflows.

## Please complete the following:

- [x] I have added screenshots for all UI updates
- [x] I confirm I have thoroughly tested these changes and take full
responsibility for any bugs introduced
- [x] I understand that submitting code with bugs that could have been
caught through manual testing blocks releases and new features for all
contributors

## Please put your Discord username so you can be contacted if a bug or
regression is found:

fake.neo

---------

Co-authored-by: Scott Anderson <662325+scottanderson@users.noreply.github.com>
2025-04-02 09:09:38 -07:00