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

Move Cloudflare Workers instructions #17

Merged
merged 2 commits into from
Nov 18, 2022
Merged

Move Cloudflare Workers instructions #17

merged 2 commits into from
Nov 18, 2022

Conversation

rudychung
Copy link
Contributor

@rudychung rudychung commented Nov 18, 2022

Addresses issue #12
Re-located the Cloudflare Workers instructions from the top of worker/index.js to CONTRIBUTING.md.

@SerpentBytes SerpentBytes added ✨ enhancement New feature or request 🗒️ documentation Improvements or additions to documentation and removed ✨ enhancement New feature or request labels Nov 18, 2022
CONTRIBUTING.md Outdated

1. Run `npx wrangler dev src/index.js` in your terminal to start a development server
Copy link
Collaborator

@SerpentBytes SerpentBytes Nov 18, 2022

Choose a reason for hiding this comment

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

You could make it more generic: npx wrangler dev <filepath> in case we decide to add more workers later.

CONTRIBUTING.md Show resolved Hide resolved
Copy link
Collaborator

@SerpentBytes SerpentBytes left a comment

Choose a reason for hiding this comment

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

I have left some feedback.

@rudychung rudychung requested review from SerpentBytes and removed request for humphd, CDPhu, batunpc and saminarp November 18, 2022 20:44
Copy link
Collaborator

@SerpentBytes SerpentBytes left a comment

Choose a reason for hiding this comment

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

@rudychung Thank you.

@rudychung
Copy link
Contributor Author

rudychung commented Nov 18, 2022

Whoops, didn't mean to remove the other requested reviewers. Could you add them back

@SerpentBytes
Copy link
Collaborator

Whoops, didn't mean to remove the other requested reviewers. Could you add them back

Done.

Copy link
Collaborator

@batunpc batunpc left a comment

Choose a reason for hiding this comment

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

Thank you @rudychung ! please squash the commits before merging

@SerpentBytes SerpentBytes merged commit 25566a9 into Seneca-CDOT:main Nov 18, 2022
@rudychung rudychung deleted the issue-12 branch December 6, 2022 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗒️ documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants