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

docs: fix copyrights based on git log #557

Conversation

danielpeintner
Copy link
Member

No description provided.

@danielpeintner
Copy link
Member Author

@erossignon
Copy link
Contributor

I have spotted copyright not handled by the script in /.vscode/settings.json

@relu91
Copy link
Member

relu91 commented Oct 11, 2021

I have spotted copyright not handled by the script in /.vscode/settings.json

Good point, but I think it is from an old configuration for a vs code extension. @danielpeintner should we keep it? do you recall the extension name? if you do we can add a configuration to let vs code warn users that we suggest such extension when developing node-wot.

@danielpeintner
Copy link
Member Author

I have spotted copyright not handled by the script in /.vscode/settings.json

Good point, but I think it is from an old configuration for a vs code extension. @danielpeintner should we keep it? do you recall the extension name? if you do we can add a configuration to let vs code warn users that we suggest such extension when developing node-wot.

Thanks for spotting!

This is somewhat difficult to deal with because for new files it should contain the current year only.
Copyright (c) 2021 Contributors to the Eclipse Foundation

I agree with @relu91 I am not sure if it is still in use. I don't think so.
The extension is most likely https://marketplace.visualstudio.com/items?itemName=doi.fileheadercomment

Anyhow, I think we should remove it. So did I in my last commit.
If anyone feels strongly we can still bring it back.

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.

3 participants