Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove more variables found by oxlint no-unused-vars #1792

Merged
merged 1 commit into from
Aug 30, 2024

Conversation

jelly
Copy link
Member

@jelly jelly commented Aug 29, 2024

No description provided.

Copy link

We were not able to find or create Copr project packit/cockpit-project-cockpit-machines-1792 specified in the config with the following error:

Cannot create a new Copr project (owner=packit project=cockpit-project-cockpit-machines-1792 chroots=['centos-stream-10-x86_64', 'centos-stream-9-x86_64']): Copr: 'packit/cockpit-project-cockpit-machines-1792' already exists. Copr HTTP response is 400 BAD REQUEST.

Unless the HTTP status code above is >= 500, please check your configuration for:

  1. typos in owner and project name (groups need to be prefixed with @)
  2. whether the project name doesn't contain not allowed characters (only letters, digits, underscores, dashes and dots must be used)
  3. whether the project itself exists (Packit creates projects only in its own namespace)
  4. whether Packit is allowed to build in your Copr project
  5. whether your Copr project/group is not private

@martinpitt
Copy link
Member

martinpitt commented Aug 29, 2024

103:26 error 'nodeDevices' is assigned a value but never used no-unused-vars

Give me just ... one more! 🎶

Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cheers! 🧹

@jelly jelly merged commit 23100b9 into cockpit-project:main Aug 30, 2024
27 checks passed
@jelly jelly deleted the oxlint-no-unused-vars branch August 30, 2024 11:23
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