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

update screeps dep. to safe version #17

Closed
wants to merge 1 commit into from

Conversation

CarsonBurke
Copy link
Contributor

No description provided.

@CarsonBurke
Copy link
Contributor Author

@Jomik
previous versions of the server have major security issues that this update should resolve.
I have not tested this yet.

@CarsonBurke
Copy link
Contributor Author

CarsonBurke commented Jan 12, 2024

I have now tested it and had no issues.
also, any chance I could have some permissions over this repo to help maintain it?

@Jomik
Copy link
Owner

Jomik commented Jan 13, 2024

The package.json actually does not currently control what is used when building this image. I intended to have it as dependency and then use something like dependabot to trigger updates automatically.
I will look at setting that up now so that it happens automatically.

@Jomik Jomik force-pushed the main branch 4 times, most recently from 67cf8fb to 2ad90e7 Compare January 13, 2024 16:50
@Jomik
Copy link
Owner

Jomik commented Jan 13, 2024

I have now tested it and had no issues. also, any chance I could have some permissions over this repo to help maintain it?

Thanks! I am unsure what maintenance you are thinking of? Version updates should now be automated using dependabot. Development itself I would still like to be core part of, so if you have any suggestions or changes there, I would prefer it go through a PR so that I can scrutinize it first 😄

@Jomik
Copy link
Owner

Jomik commented Jan 13, 2024

Fixed in #18

@Jomik Jomik closed this Jan 13, 2024
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.

2 participants