Skip to content

Commit

Permalink
ci: scope restricted-syntax to exported code
Browse files Browse the repository at this point in the history
They're more important and the terminology warnings in tests were hiding more substantial warnings.

In inter-protocol this cut  535 terminology warnings in tests, leaving 180 in produc/exported code
  • Loading branch information
turadg committed Apr 5, 2023
1 parent faadb30 commit 4630a1f
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ module.exports = {
// CI has a separate format check but keep this warn to maintain that "eslint --fix" prettifies
// UNTIL https://github.com/Agoric/agoric-sdk/issues/4339
'prettier/prettier': 'warn',
'no-restricted-syntax': ['warn', ...deprecatedTerminology],
},
settings: {
jsdoc: {
Expand Down Expand Up @@ -83,6 +82,8 @@ module.exports = {
// are code-reviewed.
// TODO upgrade this to 'error'
'@jessie.js/no-nested-await': 'warn',
// TODO upgrade this (or a subset) to 'error'
'no-restricted-syntax': ['warn', ...deprecatedTerminology],
},
},
{
Expand Down

0 comments on commit 4630a1f

Please sign in to comment.