Skip to content

Conversation

@ericcornelissen
Copy link
Contributor

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.

Description

Fix the "Using a custom configuration" link the README. The actual ID of the header on the page is #using-a-custom-configuration-file, so I updated both the link and the link title to match. (as far as I can tell all other links in the README work as expected.)


As an aside (not sure where best to report this), the snippet for Specifying directories to scan in the docs seems to be outdated. paths-ignore and paths can no longer be specified in the with: statement, according to this error message:

##[warning]Unexpected input(s) 'paths-ignore', valid inputs are ['tools', 'languages', 'token', 'matrix', 'config-file', 'queries']

Fix the "Using a custom configuration" link the README. The actual ID of the header on the page is "using-a-custom-configuration-file", so I updated both the link and the link test to match. (as far as I can tell all other links in the README work as expected)
@robertbrignull
Copy link
Contributor

Thanks for spotting that and fixing it 👍

Thanks for the point about "Specifying directories to scan" too. I've forwarded this onto the docs team and we'll try to get that page fixed as soon as possible.

@robertbrignull
Copy link
Contributor

We can ignore that one job failure. There's no way this PR broke it. I think that job just doesn't work on PRs from forks, but we'll look into it.

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.

2 participants