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.

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

MSAL-Node Tests: Fixed file extensions on all imports & added MS license to all files #7454

Merged
merged 6 commits into from
Dec 21, 2024
Merged

MSAL-Node Tests: Fixed file extensions on all imports & added MS license to all files #7454

merged 6 commits into from
Dec 21, 2024

Conversation

Robbie-Microsoft
Copy link
Collaborator

Addressed tech debt in the msal-node tests.

  1. Added ".js" to all file names in imports
  2. Added the MS license at the top of all files

@github-actions github-actions bot added the msal-node Related to msal-node package label Dec 6, 2024
@Robbie-Microsoft Robbie-Microsoft merged commit 958da50 into dev Dec 21, 2024
4 of 8 checks passed
@Robbie-Microsoft Robbie-Microsoft deleted the import_file_extension_fixes branch December 21, 2024 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gladjohn gladjohn gladjohn approved these changes

@bgavrilMS bgavrilMS bgavrilMS approved these changes

@sameerag sameerag Awaiting requested review from sameerag sameerag is a code owner

@tnorling tnorling Awaiting requested review from tnorling tnorling is a code owner

@hectormmg hectormmg Awaiting requested review from hectormmg hectormmg is a code owner

@peterzenz peterzenz Awaiting requested review from peterzenz peterzenz is a code owner

Assignees
No one assigned
Labels
msal-node Related to msal-node package
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Robbie-Microsoft @bgavrilMS @gladjohn