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

web230.js.org #3522

Merged
merged 2 commits into from Nov 9, 2019

Conversation

@paylesworth
Copy link
Contributor

paylesworth commented Nov 8, 2019

Copy link
Member

MattIPv4 left a comment

Please only add your new subdomain request to the file. All these other formatting changes are not wanted here.

@MattIPv4 MattIPv4 added the invalid label Nov 9, 2019
@paylesworth

This comment has been minimized.

Copy link
Contributor Author

paylesworth commented Nov 9, 2019

I was sure I had shut off auto formatting on my editor but I guess not. I have fixed it and checked the diff. This is my first pull request. You must get a lot of newbie mistakes like this. Sorry and thanks for your patience.

Copy link
Member

MattIPv4 left a comment

Commit valid, CNAME present, site appears to have content *

* based off https://github.com/paylesworth/web230

@MattIPv4 MattIPv4 added add project user and removed invalid labels Nov 9, 2019
@MattIPv4 MattIPv4 changed the title added web230 web230.js.org Nov 9, 2019
@indus

This comment has been minimized.

Copy link
Member

indus commented Nov 9, 2019

@indus indus merged commit 539953d into js-org:master Nov 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.