Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Change paths for js #81

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 18, 2023
Merged

Change paths for js #81

merged 1 commit into from
Apr 18, 2023

Conversation

digimangos
Copy link
Contributor

@digimangos digimangos commented Apr 17, 2023 •
edited
Loading

Updating codeql.yml following refactor which is now creating js files instead of mjs.

@digimangos digimangos added the bug Something isn't working label Apr 17, 2023
@digimangos digimangos requested a review from a team as a code owner April 17, 2023 22:57
@digimangos digimangos self-assigned this Apr 17, 2023
@digimangos digimangos requested a review from stoe as a code owner April 17, 2023 22:57
@digimangos
Copy link
Contributor Author

@stoe, just noticed that utils\ActionPolicy.mjs is still using the mjs extension, do you anticipate to change this:

  1. To js also
  2. Or should the codeql.yml file include both mjs and js?

@stoe
Copy link
Contributor

stoe commented Apr 18, 2023

@digimangos utils/ActionPolicy.mjs should change as well.

@stoe stoe enabled auto-merge (squash) April 18, 2023 00:15
@stoe stoe merged commit 840ba98 into main Apr 18, 2023
@stoe stoe deleted the update-codeql-config branch April 18, 2023 00:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stoe stoe stoe approved these changes

Assignees

@digimangos digimangos

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@digimangos @stoe