Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

(MERGE WHEN READY TO PUBLISH SITE) Publish the site to iceberg.apache.org #36

Merged
merged 2 commits into from
Feb 6, 2022
Merged

(MERGE WHEN READY TO PUBLISH SITE) Publish the site to iceberg.apache.org #36

merged 2 commits into from
Feb 6, 2022

Conversation

samredai
Copy link
Contributor

@samredai samredai commented Feb 5, 2022

This updates the .asf.yaml file to add the publish directive to https://iceberg.apache.org.

kbendick
kbendick approved these changes Feb 6, 2022
.asf.yaml Outdated
@@ -33,4 +33,5 @@ notifications:
issues: [email protected]
pullrequests: [email protected]


publish:
whoami: asf-site
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: a newline so we don’t get the red mark? I’m not sure how much people care but I don’t think see them in the main repo.

Copy link
Contributor

@kbendick kbendick left a comment

Choose a reason for hiding this comment

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

This LGTM. Thanks @samredai!

@rdblue rdblue merged commit 29e7b32 into apache:main Feb 6, 2022
@rdblue
Copy link
Contributor

rdblue commented Feb 6, 2022

Thanks, @samredai! Great work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants