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

Add towncrier #618

Merged
merged 3 commits into from
Oct 1, 2018
Merged

Add towncrier #618

merged 3 commits into from
Oct 1, 2018

Conversation

Zlopez
Copy link
Contributor

@Zlopez Zlopez commented Sep 26, 2018

No description provided.

@codecov-io
Copy link

codecov-io commented Sep 26, 2018

Codecov Report

Merging #618 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #618   +/-   ##
=======================================
  Coverage   90.03%   90.03%           
=======================================
  Files          55       55           
  Lines        2650     2650           
  Branches      343      343           
=======================================
  Hits         2386     2386           
  Misses        201      201           
  Partials       63       63
Impacted Files Coverage Δ
anitya/app.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 711e101...be2bf48. Read the comment docs.

* ``PR42`` when the change has been implemented in pull request ``42``, and
there is no associated issue
* ``Cabcdef`` when the change has been implemented in changeset ``abcdef``, and
there is no associated issue or pull request.

Choose a reason for hiding this comment

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

Since you use mergify and it rebases, I recommend against using commit hashes - the author won't be able to predict their hash, and so any hashes they use will lead to a 404 link.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, you are right. I didn't noticed this.

* ``Cabcdef`` when the change has been implemented in changeset ``abcdef``, and
there is no associated issue or pull request.
* ``username`` for contributors (``author`` extention). It should be the
username part of their commits' email address.

Choose a reason for hiding this comment

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

s/commits'/commit's/

@@ -0,0 +1,19 @@
#!/usr/bin/env python3

Choose a reason for hiding this comment

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

I recommend adding a docblock that explains the purpose of this script and any args it accepts (if any).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Most of the code is copied from fedora-infra/fedora-messaging#67.
But I will add it, so this is more readable.

@@ -0,0 +1,19 @@
#!/usr/bin/env python3

Choose a reason for hiding this comment

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

I didn't see a general Copyright claim on Anitya - I also recommend adding a copyright claim on this file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same as above.

Copy link

@bowlofeggs bowlofeggs left a comment

Choose a reason for hiding this comment

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

w00t

@mergify mergify bot merged commit 221fee2 into fedora-infra:master Oct 1, 2018
@Zlopez Zlopez deleted the towncrier branch October 8, 2018 08:40
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