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

Small fixes to source generator #3466

Merged
merged 5 commits into from
Oct 31, 2023
Merged

Small fixes to source generator #3466

merged 5 commits into from
Oct 31, 2023

Conversation

papafe
Copy link
Contributor

@papafe papafe commented Oct 30, 2023

This PR does two things:

  • Raises an error when running the source generator against c# code with a version lower than 8.0
  • Removes the use of not from the generated code, so it should be possible to use c# 8.0 as the minimum supported version. (most of the changes in the PR are due to this)

Fixes #3265

TODO

  • Changelog entry

@coveralls-official
Copy link

Pull Request Test Coverage Report for Build 6693702570

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 80.176%

Totals Coverage Status
Change from base Build 6463545805: 0.0%
Covered Lines: 6320
Relevant Lines: 7775

💛 - Coveralls

@papafe papafe merged commit 10aaf6a into main Oct 31, 2023
76 of 80 checks passed
@papafe papafe deleted the fp/fix-sg branch October 31, 2023 08:42
nirinchev added a commit that referenced this pull request Nov 1, 2023
* main:
  Small fixes to source generator (#3466)
  Make websocket error logging exceptionally verbose (#3459)
  Emit errors for collection assignments (#3456)
  Fix docfx source files (#3453)
  Fix typo (#3454)
  Add User.Changed event (#3433)
  Allow customizing json ignore attribute for serialized classes (#3451)
  Update README.md (#3450)
  Fix changelog
  Prepare for vNext (#3444)
  Prepare for 11.5.0 (#3442)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue an error when using the source generators with older versions of C#
2 participants