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.

Sign up for GitHub

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

Jump to bottom

Add Nuget Config, Bazel and EdgeQL Support, Fix Output Formatter #999

Merged
merged 13 commits into from
May 24, 2023
Merged

Add Nuget Config, Bazel and EdgeQL Support, Fix Output Formatter #999

merged 13 commits into from
May 24, 2023

Conversation

Embers-of-the-Fire
Copy link
Contributor

@Embers-of-the-Fire Embers-of-the-Fire commented May 20, 2023 •
edited
Loading

Changes:

  • Add support for NuGet Configs, using xml parser but as a new language. Issue#976
  • Fix CONTRIBUTING.md's Bug Report part.
  • Add language support for Bazel. Issue#898
  • Add language support for EdgeQL(EdgeDB). Issue#940
  • Fix output formatter, now all names are serialized using LanguageType::name. Issue#975

@Embers-of-the-Fire Embers-of-the-Fire changed the title Add Nuget Configuration Support Add Nuget Configuration and Bazel Support, Fix Output Formatter May 20, 2023
@Embers-of-the-Fire
Copy link
Contributor Author

I've add serde(alias) into tera templates, and I hope this won't lead to other problems. So when parsing data, now the two types of name are both supported, and when serializing I only use the human friendly variant.

If it's not a good idea, ping me and I'll change my solution.

@Embers-of-the-Fire Embers-of-the-Fire changed the title Add Nuget Configuration and Bazel Support, Fix Output Formatter Add Nuget Config, Bazel and EdgeQL Support, Fix Output Formatter May 20, 2023
XAMPPRocky
XAMPPRocky reviewed May 20, 2023
View reviewed changes
languages.json Outdated Show resolved Hide resolved
@XAMPPRocky
Copy link
Owner

Thank you for your PR, and congrats on your first contribution! 🎉

@XAMPPRocky XAMPPRocky merged commit 21be04c into XAMPPRocky:master May 24, 2023
Embers-of-the-Fire added a commit to Embers-of-the-Fire/tokei that referenced this pull request Jun 1, 2023
@Embers-of-the-Fire
Merge pull request #4 from XAMPPRocky/master
069fe69 
Add Nuget Config, Bazel and EdgeQL Support, Fix Output Formatter (XAMPPRocky#999)
Embers-of-the-Fire added a commit to Embers-of-the-Fire/tokei that referenced this pull request Jun 8, 2023
@Embers-of-the-Fire
Add Nuget Config, Bazel and EdgeQL Support, Fix Output Formatter (XAM…
b0ed177 
…PPRocky#999) (#7)

* add nuget configuration support

* fix test file

* add bazel language

* fix output format

* add languages to readme

* fix serialize

* add edgeql

* edit readme

* add test file

* fix edgedb
@github-actions github-actions bot mentioned this pull request Mar 4, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@XAMPPRocky XAMPPRocky Awaiting requested review from XAMPPRocky

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Embers-of-the-Fire @XAMPPRocky