Skip to content

Conversation

@Panquesito7
Copy link
Member

Things added/changed:

  • Improve contributing deprecation notice.

Copy link
Collaborator

@Snailedlt Snailedlt left a comment

Choose a reason for hiding this comment

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

Looks good, but for some reason there seems to be an empty change at line 506. Any idea what that is?

| vscode.dev | https://vscode.dev/ | Visual Studio Code in the browser |
| Iloveimg | https://www.iloveimg.com/resize-image/resize-svg | Resizing SVG's |
| svgviewer.dev | https://www.svgviewer.dev/ | View, save, and optimize SVGs |
| svgviewer.dev | https://www.svgviewer.dev/ | View, save, and optimize SVGs |
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't see a change here. Could you point out what exactly changed on this line?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not sure about that, but I didn't make any changes there.

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'll approve once this diff is fixed. Invisible changes always leave me with a bad taste in the mouth

| vscode.dev | https://vscode.dev/ | Visual Studio Code in the browser |
| Iloveimg | https://www.iloveimg.com/resize-image/resize-svg | Resizing SVG's |
| svgviewer.dev | https://www.svgviewer.dev/ | View, save, and optimize SVGs |
| svgviewer.dev | https://www.svgviewer.dev/ | View, save, and optimize SVGs |
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'll approve once this diff is fixed. Invisible changes always leave me with a bad taste in the mouth

@Panquesito7
Copy link
Member Author

I found the issue: it seems there was a missing endline in the original file, and on this file, it was automatically added.
That's OK as some IDEs could have issues without an endline. There should be no issues with that. 🙂

@Snailedlt Snailedlt merged commit 87b764f into develop Sep 13, 2022
@Panquesito7 Panquesito7 deleted the update_contributing branch September 13, 2022 14:42
Snailedlt pushed a commit to Snailedlt/devicon that referenced this pull request Oct 15, 2022
Snailedlt pushed a commit to Snailedlt/devicon that referenced this pull request Oct 15, 2022
Snailedlt pushed a commit to Snailedlt/devicon that referenced this pull request Oct 15, 2022
@Snailedlt Snailedlt mentioned this pull request Feb 5, 2024
GCHQDeveloper926 pushed a commit to GCHQDeveloper926/devicon that referenced this pull request Dec 20, 2024
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.

4 participants