Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

Remove redundant config #8

Closed
wants to merge 1 commit into from
Closed

Conversation

ChrisBAshton
Copy link
Contributor

@ChrisBAshton ChrisBAshton commented Jan 18, 2021

The sassLint option was removed in alphagov/govuk-jenkinslib#80
And the Yarn install step is unnecessary as of alphagov/govuk-jenkinslib#82
The yarn run lint step should still run after yarn dependencies have been installed.
And the rubyLintDiff option was removed in alphagov/govuk-jenkinslib#57.


Search page examples to sanity check:

  • https://[HEROKU-APP-ID].herokuapp.com/search/all
  • https://[HEROKU-APP-ID].herokuapp.com/search/research-and-statistics
  • https://[HEROKU-APP-ID].herokuapp.com/search/news-and-communications?parent=%2Feducation&topic=c58fdadd-7743-46d6-9629-90bb3ccc4ef0
  • https://[HEROKU-APP-ID].herokuapp.com/get-ready-brexit-check/questions
  • https://[HEROKU-APP-ID].herokuapp.com/drug-device-alerts
  • https://[HEROKU-APP-ID].herokuapp.com/find-eu-exit-guidance-business
  • https://[HEROKU-APP-ID].herokuapp.com/find-eu-exit-guidance-business?keywords=eori&order=relevance
  • https://[HEROKU-APP-ID].herokuapp.com/find-eu-exit-guidance-business?sector_business_area%5B%5D=aerospace&order=topic
  • https://[HEROKU-APP-ID].herokuapp.com/uk-nationals-living-eu

Other finders

Jenkinsfile Outdated Show resolved Hide resolved
The sassLint option was removed in alphagov/govuk-jenkinslib#80
And the Yarn install step is unnecessary as of alphagov/govuk-jenkinslib#82
The `yarn run lint` step should still [run after yarn dependencies have been installed](https://github.com/alphagov/govuk-jenkinslib/pull/82/files#diff-6f9c4dd7a8fcf2fc03dc9947f9c2253a3d3d640daf5d986943e4f1d1dfca306bR191).
And the rubyLintDiff option was removed in alphagov/govuk-jenkinslib#57.
@sihugh
Copy link
Contributor

sihugh commented Apr 30, 2021

We're archiving this repo, so I'll close this PR. Thanks anyway @ChrisBAshton

@sihugh sihugh closed this Apr 30, 2021
@sihugh sihugh deleted the ChrisBAshton-patch-1 branch April 30, 2021 08:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants