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

Remove donations requests #72

Merged
merged 6 commits into from
Sep 25, 2019
Merged

Remove donations requests #72

merged 6 commits into from
Sep 25, 2019

Conversation

rprouse
Copy link
Member

@rprouse rprouse commented Sep 24, 2019

Fixes #43
Fixes #52

We received a donation offer today, so I decided to remove the requests for donations from the front page because we aren't currently accepting them. In the process, I also added @mikkelbu to the core team and did some other cleanup and updating.

I'm working on a new computer and didn't want to install Ruby, so I added Docker support to the project instead. See the Readme.md for more info.

Lastly, I can't abide by the security warnings on this repo, so I fixed those too.

Sorry, this is a larger PR, but I wanted to get it all knocked off tonight.

@@ -0,0 +1,38 @@
FROM ruby:2.6.3-buster
Copy link
Member Author

Choose a reason for hiding this comment

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

A pretty standard Dockerfile for a Ruby development container. VS Code will mount the source into the container and expose the ports.

@@ -0,0 +1,16 @@
{
Copy link
Member Author

Choose a reason for hiding this comment

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

This is for VS Code only. Tells VS Code to open the project in a Docker container and configures it. Ignored otherwise. See the readme.

@@ -0,0 +1,3 @@
* text=auto eol=lf
Copy link
Member Author

Choose a reason for hiding this comment

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

Consistency for editing in a Linux environment.

@@ -1,6 +1,7 @@
source "https://rubygems.org"

gem 'jekyll'
gem 'github-pages'
gem 'jekyll', '>= 3.8.5'
Copy link
Member Author

Choose a reason for hiding this comment

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

Fixes the security issues.

@@ -1,12 +1,58 @@
# NUnit.org website

## Installing Jekyll
To get started developing the website, you have two options. You can develop in a Docker Container using VS Code, or you can install Ruby locally and develop locally.
Copy link
Member Author

Choose a reason for hiding this comment

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

A lot more words, but a very nice development experience. I ❤️ Docker.

atom_path: /atom.xml
host: 0.0.0.0
Copy link
Member Author

Choose a reason for hiding this comment

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

Need to listen on all addresses so that we can forward the port from the Docker container to the host

Copy link
Member

@mikkelbu mikkelbu left a comment

Choose a reason for hiding this comment

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

I only have a few comments as I know very little about Ruby, "Visual Studio Code Remote Development", and Jekyll, but as far as I can tell is looks correct. So I'll "approve with suggestions" 😄.

Should we also remove the file _includes\paypal.html ?

EDIT: Ps. And I look forward to try this out as I've not had the energy to install Ruby etc. just to solve e.g. #50. Now it sounds much more manageable (and I get to try out "Remote Development" in VS Code).

donations.md Outdated Show resolved Hide resolved
donations.md Outdated Show resolved Hide resolved
@rprouse
Copy link
Member Author

rprouse commented Sep 25, 2019

Thanks for the suggestions @mikkelbu, I've done them all. I think you will like the Docker experience in VS Code. It is very slick. I've been adding support to quite a few projects I work on and really like it. Maybe do NUnit next? 😄

@rprouse rprouse merged commit d81f2b8 into master Sep 25, 2019
@rprouse rprouse deleted the rob/security-fixes branch September 25, 2019 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Nokogiri dependency Remove or update donations link
2 participants