Skip to content
This repository has been archived by the owner on May 4, 2024. It is now read-only.

fix: move demo.js to the docs folder #155

Merged
merged 1 commit into from
Mar 10, 2022
Merged

fix: move demo.js to the docs folder #155

merged 1 commit into from
Mar 10, 2022

Conversation

XhmikosR
Copy link
Contributor

@XhmikosR XhmikosR commented Mar 10, 2022

npm run lint will complain, though, so alternatively the file could be moved to a new folder or the docs folder.

Note that currently the file is linked in README.md from the root folder.

@XhmikosR XhmikosR requested a review from a team as a code owner March 10, 2022 12:03
@wraithgar
Copy link
Member

Yes it is going to have to move to a docs folder, and the readme should be fixed too.

@wraithgar
Copy link
Member

This will also need to be a fix since it does affect the published module.

@XhmikosR
Copy link
Contributor Author

Done:

C:\Users\xmr\Desktop\gauge>npm i --production

added 11 packages, and audited 12 packages in 11s

found 0 vulnerabilities

C:\Users\xmr\Desktop\gauge>node docs\demo.js
Demoing output for ASCII
Demoing output for colorASCII
Demoing output for brailleSpinner
Demoing output for colorBrailleSpinner

@XhmikosR XhmikosR changed the title chore: move demo.js to the root folder fix: move demo.js to the docs folder Mar 10, 2022
@wraithgar wraithgar merged commit 1faf8cf into npm:main Mar 10, 2022
@XhmikosR XhmikosR deleted the move-demo branch March 10, 2022 16:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants