Skip to content

Conversation

@NadeemYaseen
Copy link
Contributor

Motivate of the pull request

  • [ x] To address an existing issue. If so, please provide a link to the issue:
  • Breaking new feature. If so, please describe details in the description part.

Describe the technical details

CI is broken due to change in GitHub action runner version so fixing it to make it compatible with our current setup

What is currently done? (Provide issue link if applicable)

What does this pull request change?

Which part of the code base require a change

  • Build compatibility
  • Documentation
  • Regression tests
  • [x ] Continous Integration (CI) scripts

Impact of the pull request

  • Require a change on Quality of Results (QoR)
  • Break back-compatibility. If so, please list who may be influenced.

@NadeemYaseen
Copy link
Contributor Author

NadeemYaseen commented Oct 16, 2024

The actual error was to add these lines in patched VPR CMake.
Again we fell into the same trap, Xifin brought in large changes from the VPR submodule and our copying did not go well.
It is fixed PR #894

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants