Skip to content

Comments

feat(linter): support user-configurable secrets for oxc-security/api-keys#5938

Merged
graphite-app[bot] merged 1 commit intomainfrom
don/09-20-feat_linter_support_user-configurable_secrets_for_oxc-security_api-keys_
Oct 28, 2024
Merged

feat(linter): support user-configurable secrets for oxc-security/api-keys#5938
graphite-app[bot] merged 1 commit intomainfrom
don/09-20-feat_linter_support_user-configurable_secrets_for_oxc-security_api-keys_

Conversation

@DonIsaac
Copy link
Contributor

No description provided.

@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 20, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

Copy link
Contributor Author

DonIsaac commented Sep 20, 2024

@DonIsaac DonIsaac added the C-enhancement Category - New feature or request label Sep 20, 2024 — with Graphite App
@github-actions github-actions bot added the A-linter Area - Linter label Sep 20, 2024
@codspeed-hq
Copy link

codspeed-hq bot commented Sep 20, 2024

CodSpeed Performance Report

Merging #5938 will not alter performance

Comparing don/09-20-feat_linter_support_user-configurable_secrets_for_oxc-security_api-keys_ (1691cab) with main (4b450cc)

Summary

✅ 30 untouched benchmarks

@DonIsaac DonIsaac force-pushed the don/09-20-refactor_macros_clean_up_declare_oxc_secret branch from bb083e8 to 5be8cd9 Compare September 20, 2024 20:36
@DonIsaac DonIsaac force-pushed the don/09-20-feat_linter_support_user-configurable_secrets_for_oxc-security_api-keys_ branch from 3a8b3f4 to 1e16924 Compare September 20, 2024 20:36
@DonIsaac DonIsaac force-pushed the don/09-20-refactor_macros_clean_up_declare_oxc_secret branch from 5be8cd9 to 3015a91 Compare September 25, 2024 17:15
@DonIsaac DonIsaac force-pushed the don/09-20-feat_linter_support_user-configurable_secrets_for_oxc-security_api-keys_ branch from 1e16924 to e43c129 Compare September 25, 2024 17:15
@DonIsaac DonIsaac force-pushed the don/09-20-refactor_macros_clean_up_declare_oxc_secret branch from 652e8b8 to 87f3b8d Compare October 16, 2024 20:10
Base automatically changed from don/09-20-refactor_macros_clean_up_declare_oxc_secret to main October 16, 2024 20:15
@DonIsaac DonIsaac force-pushed the don/09-20-feat_linter_support_user-configurable_secrets_for_oxc-security_api-keys_ branch from e43c129 to 1b61b82 Compare October 16, 2024 20:15
@DonIsaac DonIsaac marked this pull request as ready for review October 16, 2024 20:17
@DonIsaac DonIsaac requested review from camc314 and camchenry October 16, 2024 20:18
@DonIsaac DonIsaac force-pushed the don/09-20-feat_linter_support_user-configurable_secrets_for_oxc-security_api-keys_ branch from 04b5429 to b48086f Compare October 28, 2024 18:50
@DonIsaac DonIsaac added the 0-merge Merge with Graphite Merge Queue label Oct 28, 2024
Copy link
Contributor Author

DonIsaac commented Oct 28, 2024

Merge activity

  • Oct 28, 2:53 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Oct 28, 2:59 PM EDT: A user merged this pull request with the Graphite merge queue.

@DonIsaac DonIsaac force-pushed the don/09-20-feat_linter_support_user-configurable_secrets_for_oxc-security_api-keys_ branch from b48086f to 1691cab Compare October 28, 2024 18:55
@graphite-app graphite-app bot merged commit 1691cab into main Oct 28, 2024
@graphite-app graphite-app bot deleted the don/09-20-feat_linter_support_user-configurable_secrets_for_oxc-security_api-keys_ branch October 28, 2024 18:59
@oxc-bot oxc-bot mentioned this pull request Nov 3, 2024
Boshen added a commit that referenced this pull request Nov 3, 2024
## [0.11.0] - 2024-11-03

- 9fd9f4f linter: [**BREAKING**] Sync sindresorhus/globals; removed
Object.prototype properties from builtin and es* globals (#6991)
(Boshen)

### Features

- 6b619da editor: Listen to config file changes and trigger a
didChangeConfiguration update (#6964) (Nicholas Rayburn)
- 7872927 editor/vscode: Support window/showMessage event (#7085)
(Nicholas Rayburn)
- 2184588 linter: Do not bail for unmatched rules yet (#7093) (Boshen)
- a6fcd81 linter: Add `import/no-commonjs` rule (#6978) (Dmitry
Zakharov)
- 1691cab linter: Support user-configurable secrets for
`oxc-security/api-keys` (#5938) (DonIsaac)
- 610621c linter: Implement `react/style-prop-object` (#6342) (Albert
Kaaman)
- 1e2f012 linter: Add `oxc/no-map-spread` (#6751) (DonIsaac)
- 1c66473 linter: Implement `eslint/prefer-object-has-own` (#6905)
(tomoya yanagibashi)

### Bug Fixes

- ebf3753 editor: Fix onConfigChange to send the correct config for
didChangeConfiguration notification (#6962) (Nicholas Rayburn)
- 79bf74a linter: Check is_reference_to_global_variable in
`no-array-constructor` (#7067) (Naoya Yoshizawa)
- 38d1f78 linter: Remove confusing help text for now (#7081) (Cam
McHenry)
- 147e2e4 linter: Allow replacing rule when none are enabled yet (#7014)
(camchenry)
- 7aa496a linter: Remove unsafe fixer of `no-useless-spread` (#6655)
(dalaoshu)
- f5a7134 linter/no-unused-vars: False positive for discarded reads
within sequences (#6907) (DonIsaac)

### Documentation

- 4551baa linter: Document `rules` (#6983) (Boshen)

### Refactor

- a8dc75d linter: Remove unused CLI result types (#7088) (camchenry)
- 8f1460e linter: Move `LintPlugins` from `LintOptions` to `LintConfig`
(#6932) (DonIsaac)

### Testing

- c35d3f2 linter: Improve test failure output (#6975) (camchenry)

---------

Co-authored-by: Boshen <[email protected]>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0-merge Merge with Graphite Merge Queue A-linter Area - Linter C-enhancement Category - New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants