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

[WIP] chore: Add workflows, update README #4

Merged
merged 15 commits into from
Apr 7, 2021

Conversation

ebebbington
Copy link
Member

@ebebbington ebebbington commented Apr 7, 2021

Fixes #1

Summary

  • Brings workflows and README inline with rest of the org

TODO

  • <script src="https://unpkg.com/@drashland/moogle@0.0.1-alpha-6"></script> doesn't work
  • Add tests to ensure the above does work if possible
  • There is a lot of ci jobs, maybe we could target latest of node?

@ebebbington ebebbington added the Type: Chore Merging this pull request results in a patch version increment label Apr 7, 2021
@ebebbington
Copy link
Member Author

using https://unpkg.com/@drashland/moogle@0.0.1-alpha-6 doesn't seem to work

@Guergeiro
Copy link
Member

using https://unpkg.com/@drashland/moogle@0.0.1-alpha-6 doesn't seem to work

The reason is that that version was not published to npm since the workflow failed. That's why this is still "alpha", because if failing release all over the place.

0.0.1-alpha-1 was the last one that I was able to publish.

@Guergeiro
Copy link
Member

One can also use jsdelivery

@ebebbington
Copy link
Member Author

Alright I’ll try check it with specifying then alpha 1 version

@Guergeiro
Copy link
Member

Btw, the README.md stopped having syntax highlighting. Is it because of the extra spaces added?

@ebebbington
Copy link
Member Author

Btw, the README.md stopped having syntax highlighting. Is it because of the extra spaces added?

yeah i guess so - seems like a bug with deno fmt, but ill just 'unindent' it lol

@ebebbington
Copy link
Member Author

good catch!

@ebebbington
Copy link
Member Author

Using alpha 1 as the release, using moogle from the browser does work

@Guergeiro
Copy link
Member

That's why I had the formatter only working on ./src. Maybe we could be specific about the directories/files?

@ebebbington
Copy link
Member Author

That's why I had the formatter only working on ./src. Maybe we could be specific about the directories/files?

it's all good now, i think it's just cause the code blocks were indented and using \```

@Guergeiro
Copy link
Member

As for the node tests, I followed the ones that still are not EOL, but we could just as well just test on Active LTS and Current.

Copy link
Member

@Guergeiro Guergeiro left a comment

Choose a reason for hiding this comment

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

Let's GO!

@ebebbington ebebbington merged commit 249ee3f into master Apr 7, 2021
@ebebbington ebebbington deleted the issue-#1-consistency branch April 7, 2021 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Chore Merging this pull request results in a patch version increment
Development

Successfully merging this pull request may close these issues.

Bring moogle inline with our other repos
2 participants