Dynamo
Dynamo copied to clipboard
DYN-6651 Updating IssueTypePredicter
Purpose
Updating IssueTypePredicter workflow to use .NET 8 The next PR needs to be merged first: https://github.com/DynamoDS/IssuesTypePredicter/pull/2
Declarations
Check these if you believe they are true
- [X] The codebase is in a better state after this PR
- [X] Is documented according to the standards
- [X] The level of testing this PR includes is appropriate
- [X] User facing strings, if any, are extracted into
*.resx
files - [ ] All tests pass using the self-service CI.
- [ ] Snapshot of UI changes, if any.
- [ ] Changes to the API follow Semantic Versioning and are documented in the API Changes document.
- [ ] This PR modifies some build requirements and the readme is updated
- [ ] This PR contains no files larger than 50 MB
Release Notes
Updating IssueTypePredicter workflow to use .NET 8
Reviewers
@QilongTang @reddyashish
FYIs
UI Smoke Tests
Test: success. 2 passed, 0 failed. TestComplete Test Result Workflow Run: UI Smoke Tests Check: UI Smoke Tests - net8.0
[ContainsEmptyListOrNullTest] is failing here. So i think you did not pull in the changes from https://github.com/DynamoDS/Dynamo/pull/15190 to fix it?.
[ContainsEmptyListOrNullTest] is failing here. So i think you did not pull in the changes from #15190 to fix it?.
Thanks @reddyashish, yeah should be fine to proceed since this fork is behind master most likely