Dynamo icon indicating copy to clipboard operation
Dynamo copied to clipboard

DYN-6651 Updating IssueTypePredicter

Open RobertGlobant20 opened this issue 9 months ago • 2 comments

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

RobertGlobant20 avatar May 07 '24 00:05 RobertGlobant20

UI Smoke Tests

Test: success. 2 passed, 0 failed. TestComplete Test Result Workflow Run: UI Smoke Tests Check: UI Smoke Tests - net8.0

github-actions[bot] avatar May 07 '24 00:05 github-actions[bot]

[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?.

reddyashish avatar May 09 '24 15:05 reddyashish

[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

QilongTang avatar May 13 '24 15:05 QilongTang