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

Adjust1 405 template updates #147

Merged
merged 255 commits into from
Feb 7, 2024
Merged

Adjust1 405 template updates #147

merged 255 commits into from
Feb 7, 2024

Conversation

ldlharper
Copy link
Contributor

No description provided.

andrewrlee and others added 30 commits November 19, 2020 16:32
* Update dependencies

Also move to use declaration overrides for describing additional possible fields in session data

Also removing/moving unnecessary prod dependencies

Moving jest tests to run in parallel

* Fix types for user

* Do not allow any warnings

* Correcting test results path
Also
* Update dependencies
* Adopt standard github gitignore for node projects
* Using body typography
* Add rename script

* PR reviews

* Removing unused vars
* Decoupling auth client from redis impl
* Convert to arrow functions in utils

* Remove returns and fix prettier issues
* Adding outdated job

* Updating dependencies

* Removing patch to minor version
* Add new veracode scan for schedueled workflow

* Re-add - check_outdated job
…ks (#27)

* Updating dependencies and adding slack notifications to outdated checks

* Add slack orb
* Ensure that the `SLACK_ACCESS_TOKEN` env var is set for `check_outdated`

As the final step (on fail) is a slack notification we need to use a context containing the variable.

* Refactor the channel for alerts into parameters

This establishes a reusable pattern in case consumers wish to notify slack on other jobs.
The "medium" configuration was not an acceptable term, should be one of
"info", "low", "moderate", "high" or "critical". This meant that it
would not pick up on anything!
* Updating dependencies and node

* Use parameterised slack channel rather than default of typescript alerts channel

* Extracting out executor
andrewrlee and others added 15 commits December 18, 2023 08:00
* Update CHANGELOG.md

* Update CHANGELOG.md

---------

Co-authored-by: nishanthmahasamudram <129746957+nishanthmahasamudram@users.noreply.github.com>
Co-authored-by: Paul Solecki <51918433+psoleckimoj@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Upgrade to GovUK 5 and MoJ 2.0

* Update sass command
* Update all non major NPM dependencies
* Update Node.js to v20.11.0
* Update dependency husky to v9
* Update MoJ frontend, dropping jQuery which is no longer required
* Remove unused static assets route: `govuk_frontend_toolkit` is not currently included
* Prevent renovate from updating `engines.node` and `engines.npm` in package.json because we’d usually want to stick to an LTS version rather than move to the latest
`node_modules/govuk-frontend/dist/components/` was missing `govuk/`
before `components/`, however we don't really need both the higher-level
and lower-level paths here. For greater consistency, just using the
higher-level path to enable a single clearly-namespaced route to each
GOV.UK and MOJ Nunjucks component makes more sense. The lower-level path
is often unused on projects that use this template

Imports should therefore look like this:

```njk
{% from "govuk/components/checkboxes/macro.njk" import govukCheckboxes %}
```

... and not this:

```njk
{% from "checkboxes/macro.njk" import govukCheckboxes %}
```
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@ldlharper ldlharper requested a review from a team as a code owner February 6, 2024 13:10
@ldlharper ldlharper force-pushed the ADJUST1-405-template-updates branch from 34e4e11 to cf13d9f Compare February 6, 2024 16:56
Copy link
Contributor

@carloveo-moj carloveo-moj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ldlharper ldlharper merged commit 10f5255 into main Feb 7, 2024
4 checks passed
@ldlharper ldlharper deleted the ADJUST1-405-template-updates branch February 7, 2024 10:18
@ldlharper ldlharper restored the ADJUST1-405-template-updates branch February 7, 2024 10:19
@ldlharper ldlharper deleted the ADJUST1-405-template-updates branch February 7, 2024 10:19
ldlharper added a commit that referenced this pull request Feb 7, 2024
@ldlharper ldlharper restored the ADJUST1-405-template-updates branch February 7, 2024 10:26
@ldlharper ldlharper deleted the ADJUST1-405-template-updates branch February 7, 2024 10:28
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.