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

Updated Import Statements #2852

Closed
wants to merge 2 commits into from
Closed

Updated Import Statements #2852

wants to merge 2 commits into from

Conversation

Adrija-G
Copy link
Contributor

@Adrija-G Adrija-G commented Sep 2, 2023

I have updated the import statements in the code to use ES6 import syntax, replacing the previous CommonJS require statements. Specifically, I changed the following lines:

Replaced const assert = require('assert') with import assert from 'assert'; (CommonJS to ES6 import).
Replaced import remark from 'remark'; with import { default as remark } from 'remark'; (CommonJS to ES6 import).

These changes were made to ensure consistency with ES6 module syntax and maintain code quality.

Please review and merge this pull request when you have the chance. Thank you!

Copy link
Collaborator

@ltagliaferri ltagliaferri left a comment

Choose a reason for hiding this comment

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

No changes were made to the file 🤔

@ltagliaferri
Copy link
Collaborator

Thank you for opening this pull request. The changes requested have not been addressed over the course of a few days, so we are closing this PR due to inactivity and are marking it as invalid. If you would like to try again, please open a new pull request.

@ltagliaferri ltagliaferri added the invalid This doesn't seem right label Sep 27, 2023
@Adrija-G
Copy link
Contributor Author

Adrija-G commented Sep 27, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants