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

Adds gitignore, missing docs folder, low severity security updates #1

Merged
merged 5 commits into from
Jun 21, 2022

Conversation

rediris
Copy link
Member

@rediris rediris commented Mar 19, 2021

This PR resolves some, not all of the security issues with this repo.

It also adds a .gitignore, which effectively hides the thousands of files in the node_modules folder from the student.

For details, see https://github.com/gymnasium/tracker/issues/256

Also see

The issue with fractal is its dependency on vorpal, which hasn't been updated in 4 years.

Meanwhile, we should at least get this merged to that there's less security warnings thrown for students.

@rediris rediris added the security security-related issues label Mar 19, 2021
@andrewpmiller
Copy link
Member

image

@rediris rediris changed the title WIP: adds gitignore, missing docs folder, low severity security updates Adds gitignore, missing docs folder, low severity security updates Apr 1, 2021
@andrewpmiller
Copy link
Member

@rediris I tested this in GitPod and it appears to work fine

image

@rediris rediris merged commit ee1a4e2 into main Jun 21, 2022
@rediris rediris deleted the security-patches branch June 21, 2022 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
security security-related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants