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

[ci] mark documentation test as soft fail #36480

Merged
merged 1 commit into from
Jun 20, 2023
Merged

Conversation

aslonnie
Copy link
Collaborator

@aslonnie aslonnie commented Jun 15, 2023

It has been consistently failing since the strict mode is turned back on in cd0d3cd

This will make other test fixes easier to spot.

It has been consistently failing since the strict mode is turned back
on. This will make other test fixes easier to spot.

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
@can-anyscale
Copy link
Collaborator

@bveeramani : what do you think?

@bveeramani
Copy link
Member

Seems reasonable.

I'm concerned that we might introduce more documentation issues, but if :book: Documentation is always failing I guess that could happen anyway.

@aslonnie
Copy link
Collaborator Author

I'm concerned that we might introduce more documentation issues, but if 📖 Documentation is always failing I guess that could happen anyway.

I/we will remove the soft_fail once it turned green.

@bveeramani
Copy link
Member

@akshay-anyscale @scv119 looks like the the remaining build errors are with Serve, Workflows, and Observability. Do you think your respective teams will get a chance to address these in the near future?

Copy link
Collaborator

@can-anyscale can-anyscale left a comment

Choose a reason for hiding this comment

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

@aslonnie aslonnie changed the title Mark documentation test as soft fail. [ci] mark documentation test as soft fail Jun 20, 2023
@richardliaw richardliaw merged commit 08fa92d into master Jun 20, 2023
@richardliaw richardliaw deleted the lonnie-0615-softdoc branch June 20, 2023 17:22
arvind-chandra pushed a commit to lmco/ray that referenced this pull request Aug 31, 2023
It has been consistently failing since the strict mode is turned back
on. This will make other test fixes easier to spot.

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
Signed-off-by: e428265 <arvind.chandramouli@lmco.com>
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.

4 participants