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

Cherry-pick #202 #208

Merged

Conversation

amzn-mdamine
Copy link
Contributor

Cherry-pick #202

@khieta
Copy link
Contributor

khieta commented Sep 17, 2024

I'm surprised CI is failing, since Cedar v4.0 should be very similar to the version on main. @amzn-mdamine can you look into this?

@shaobo-he-aws
Copy link
Contributor

I'm looking into it. There seems to be some issues with the release branch.

@amzn-mdamine
Copy link
Contributor Author

I have verified the build and tests locally, and everything is passing on my machine. However, it seems that the CI pipeline is still failing. I'll investigate further to identify the root cause of the issue and provide an update.

@khieta
Copy link
Contributor

khieta commented Sep 17, 2024

The failure may be resolved by #209 -- try merging with the latest release/4.0.x branch

@shaobo-he-aws
Copy link
Contributor

#210 should fix it.

@shaobo-he-aws
Copy link
Contributor

@amzn-mdamine could you please rebase or merge with release/4.0.x?

Signed-off-by: Mohamed Amine Ouali <mdamine@amazon.com>
@amzn-mdamine
Copy link
Contributor Author

I did the rebase.

@shaobo-he-aws shaobo-he-aws merged commit 5fa867f into cedar-policy:release/4.0.x Sep 18, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants