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

Check the digest of the init scripts #2254

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

jdneo
Copy link
Contributor

@jdneo jdneo commented Sep 29, 2022

  • prevent generating duplicated files, the temp file will only be generated when the content is changed.
  • the digest can also be used to guard the integrity of the file.

This is the follow-up of #2249

Signed-off-by: sheche sheche@microsoft.com

- prevent generating duplicated files, the temp file will only be generated
  when the content is changed.
- the digest can also be used to guard the integrity of the file.

Signed-off-by: sheche <sheche@microsoft.com>
Copy link
Contributor

@rgrunber rgrunber left a comment

Choose a reason for hiding this comment

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

Tested this out and it seems to work well for me.

@rgrunber rgrunber added this to the End October milestone Oct 4, 2022
@rgrunber rgrunber merged commit ed4a7cf into eclipse-jdtls:master Oct 4, 2022
@jdneo jdneo deleted the cs/checksum branch October 7, 2022 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants