Skip to content
New issue

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

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

Already on GitHub? # to your account

move .eslintrc.js to .eslintrc to fix issues... #1233

Merged
merged 2 commits into from
May 6, 2024
Merged

Conversation

nickboldt
Copy link
Member

What does this PR do?

move .eslintrc.js to .eslintrc to fix issues with using import in .mjs files

Signed-off-by: Nick Boldt nboldt@redhat.com

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

N/A (or see commit message above for issue number)

How to test this PR?

N/A

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

@nickboldt nickboldt requested a review from a team as a code owner May 6, 2024 12:53
@openshift-ci openshift-ci bot requested review from gashcrumb and kim-tsao May 6, 2024 12:53
@nickboldt nickboldt force-pushed the fix-module-errors branch from 0b9bd89 to 3203626 Compare May 6, 2024 13:13
nickboldt added 2 commits May 6, 2024 10:27
…s files

Signed-off-by: Nick Boldt <nboldt@redhat.com>

remove eslint-disable now that we can use it for .mjs modules

Signed-off-by: Nick Boldt <nboldt@redhat.com>
…on ./scripts/*.mjs now that the eslint config is

fixed

Signed-off-by: Nick Boldt <nboldt@redhat.com>
@nickboldt nickboldt force-pushed the fix-module-errors branch from 3203626 to 5127ba2 Compare May 6, 2024 13:27
Copy link

sonarqubecloud bot commented May 6, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@schultzp2020 schultzp2020 left a comment

Choose a reason for hiding this comment

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

/approve
/lgtm

Copy link

openshift-ci bot commented May 6, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: schultzp2020

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved label May 6, 2024
@nickboldt nickboldt merged commit 5287a75 into main May 6, 2024
5 of 7 checks passed
@nickboldt nickboldt deleted the fix-module-errors branch May 6, 2024 13:31
Copy link
Contributor

github-actions bot commented May 6, 2024

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1233!

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants