Skip to content

Conversation

@eric-murray
Copy link
Contributor

What type of PR is this?

  • correction

What this PR does / why we need it:

Commonalities pre-release r3.1 removes the AUTHENTICATION_REQUIRED error code, leaving UNAUTHENTICATED as the only valid error code when the HTTP status code is 401. This PR removes all instances of AUTHENTICATION_REQUIRED.

Which issue(s) this PR fixes:

Fixes #39

Special notes for reviewers:

This is not a breaking change

Changelog input

 release-note
 - Remove AUTHENTICATION_REQUIRED error code

Additional documentation

None

@maxl2287 maxl2287 added correction Something needs to be corrected Fall25 Meta-release Fall25 labels May 21, 2025
@eric-murray
Copy link
Contributor Author

@maxl2287
The line "new-line-at-eof": ["on", "no"] in .gherkin-lintrc is causing me issues as I edit using the github website (on Windows). What is your preferred solution to this problem?

@maxl2287
Copy link
Contributor

maxl2287 commented May 22, 2025

@maxl2287 The line "new-line-at-eof": ["on", "no"] in .gherkin-lintrc is causing me issues as I edit using the github website (on Windows). What is your preferred solution to this problem?

Hey @eric-murray,

I am surprised that this occures when there is no new line visible in the file.
I can only see it on https://github.dev/ on your branch that there is a new line.

I am workign with IntelliJ so I do not experienced this behaviour before.

Could it be an issue regarding the line-ending in Windows ? CRLF? Idk. 🤷🏻‍♂️

@eric-murray
Copy link
Contributor Author

@maxl2287
Yes, that is the issue. Unless github is told otherwise, it adopts the local configuration, and we are obviously using different local configurations. But there is no way to change this for the github web editor.

In the absence of line endings being enforced across the project (using a core.autocrlf setting), it must either be set for the repository (in a .gitattributes file), or the rule to prohibit new-line-at-eof must be removed.

See https://docs.github.com/en/get-started/git-basics/configuring-git-to-handle-line-endings

@maxl2287
Copy link
Contributor

@maxl2287 Yes, that is the issue. Unless github is told otherwise, it adopts the local configuration, and we are obviously using different local configurations. But there is no way to change this for the github web editor.

In the absence of line endings being enforced across the project (using a core.autocrlf setting), it must either be set for the repository (in a .gitattributes file), or the rule to prohibit new-line-at-eof must be removed.

See https://docs.github.com/en/get-started/git-basics/configuring-git-to-handle-line-endings

I am okay with removing this new-line-at-eof.

@eric-murray
Copy link
Contributor Author

Raised #42

@maxl2287 maxl2287 merged commit a7701f1 into camaraproject:main May 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

correction Something needs to be corrected Fall25 Meta-release Fall25

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove AUTHENTICATION_REQUIRED error code

2 participants