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

Set ghp_{pages,path} properties #138

Merged
merged 1 commit into from
Dec 27, 2023
Merged

Set ghp_{pages,path} properties #138

merged 1 commit into from
Dec 27, 2023

Conversation

Fokko
Copy link
Contributor

@Fokko Fokko commented Dec 27, 2023

In the ASF yaml. I don't think we should make it a protected branch because I think they overwrite it every time with a force push.

Resolves #137 hopefully!

In the ASF yaml. I don't think we should make it a protected branch because I think they overwrite it every time with a force push.
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks!

@Xuanwo
Copy link
Member

Xuanwo commented Dec 27, 2023

The same setup works on opendal, I'm not sure why it doesn't work here.

Copy link
Collaborator

@liurenjie1024 liurenjie1024 left a comment

Choose a reason for hiding this comment

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

Thanks!

@Fokko
Copy link
Contributor Author

Fokko commented Dec 27, 2023

@Xuanwo Probably Github Pages had been enabled through an infra ticket in the past. The .asf.yaml is not declarative, if you leave anything out it will not disable it, you need to do that explicitly.

@Xuanwo
Copy link
Member

Xuanwo commented Dec 27, 2023

@Xuanwo Probably Github Pages had been enabled through an infra ticket in the past. The .asf.yaml is not declarative, if you leave anything out it will not disable it, you need to do that explicitly.

That makes sense!

Copy link
Contributor

@amogh-jahagirdar amogh-jahagirdar left a comment

Choose a reason for hiding this comment

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

Thanks @Fokko , thanks for reviews @Xuanwo @liurenjie1024 !

@amogh-jahagirdar amogh-jahagirdar merged commit 43697b9 into main Dec 27, 2023
7 checks passed
@Fokko Fokko deleted the fd-set-ghp branch January 4, 2024 07:42
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.

doc: rust.iceberg.apache.org is not resolved
4 participants