Skip to content

Conversation

dastan-ansys
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the maintenance Package and maintenance related label May 16, 2023
@jonahrb
Copy link
Contributor

jonahrb commented May 16, 2023

Looks good, we'll need a new unstable server image with DMS nuget 021 before we merge this

@jonahrb
Copy link
Contributor

jonahrb commented May 17, 2023

Linux unstable is good, windows unstable image is still using compressed monikers, for some reason, need to push a new image

@jonahrb jonahrb changed the title Maint/remove duplicate Maint/remove duplicate id safeguarding May 17, 2023
@RobPasMue RobPasMue changed the title Maint/remove duplicate id safeguarding MAINT: remove duplicate id safeguarding May 18, 2023
@RobPasMue RobPasMue requested a review from a team as a code owner May 18, 2023 07:44
…sterComponent everywhere."

This reverts commit 664e584.
jonahrb
jonahrb previously approved these changes May 18, 2023
RobPasMue
RobPasMue previously approved these changes May 19, 2023
Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking great @jonahrb @dastan-ansys ! Thanks for the implementation =)

Let me trigger the workflow to generate new stable images

@RobPasMue RobPasMue dismissed stale reviews from jonahrb and themself via e16d920 May 19, 2023 06:31
@RobPasMue
Copy link
Member

I think that @b-matteo's push to unstable might have crashed this... re-triggering execution just in case

@RobPasMue
Copy link
Member

Yep, that's what I thought... For some reason, @b-matteo's push also caused this to stop working

@RobPasMue RobPasMue enabled auto-merge (squash) May 24, 2023 13:14
@RobPasMue RobPasMue linked an issue May 24, 2023 that may be closed by this pull request
@RobPasMue RobPasMue merged commit f604299 into main May 24, 2023
@RobPasMue RobPasMue deleted the maint/remove-duplicate-id branch May 24, 2023 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Package and maintenance related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove ObjectPath safeguards when we get new Service versions

3 participants