Skip to content
This repository was archived by the owner on Oct 17, 2025. It is now read-only.

Allow an empty commit for travis deploy#410

Merged
schalkneethling merged 1 commit into
masterfrom
travis-deploy-allow-empty-commit
Jan 11, 2018
Merged

Allow an empty commit for travis deploy#410
schalkneethling merged 1 commit into
masterfrom
travis-deploy-allow-empty-commit

Conversation

@jwhitlock

Copy link
Copy Markdown
Contributor

git commit is failing when there is no change to the generated pages, such as updating a README. Allow an empty commit (no files changed), so that the script can continue.

@welcome

welcome Bot commented Jan 11, 2018

Copy link
Copy Markdown

💖 Thanks for opening this pull request! 💖
Here is a list of things that will help get it across the finish line: - If this is a new or updated CSS interactive example, please ensure that you followed the CSS styleguide - If this is a new or updated JavaScript interactive example, please ensure that you followed the JavaScript styleguide - If your changes affects any of the steps in our contribution docs, please also make the relevant changes there.

git commit is failing when there is no change to the generated pages,
such as updating a README. Allow an empty commit (no files changed), so
that the script can continue.
@jwhitlock jwhitlock force-pushed the travis-deploy-allow-empty-commit branch from a771594 to 7870138 Compare January 11, 2018 17:24

@schalkneethling schalkneethling left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

r+ Thanks @jwhitlock

@schalkneethling schalkneethling merged commit 051fcd6 into master Jan 11, 2018
@welcome

welcome Bot commented Jan 11, 2018

Copy link
Copy Markdown

Congrats on merging your first pull request! 🎉🎉🎉

@jwhitlock jwhitlock deleted the travis-deploy-allow-empty-commit branch January 11, 2018 17:53
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.

2 participants