Saejin Heinert

Results 14 comments of Saejin Heinert

This is certainly an interesting idea -- I think that it would be quite useful, but it doesn't fit well into our current encapsulated functionality. Normally this kind of this...

Hi @gkapfham; these diffs are actually due to the fact that one of the PRs merged previously had a reworked history compared to this PR; now that @mariakimheinert rebased on...

Hi @gkapfham, GatorGradle will not be modified to support the `poetry` version of GatorGrader; I'm in the process of publishing GatorGradle 1.0.0, which will be the last release and fix...

It will no longer be possible to use GatorGradle with *newer* versions of GatorGrader; old versions of GatorGrader will still work just fine, which is why the change to GatorGradle...

As for the `gatored.org` domain, that was needed because GatorGradle's plugin id is `org.gatored.GatorGradle`. I'd like to not change that ID, to try and keep some backward-compatibility and ease-of-upgrading. I...

@baldeosinghm @yeej2 I cannot merge this PR until the problem I mentioned in my earlier review is addressed, and Travis CI passes. Could you finish these tasks by Wednesday morning?...

I also think this can probably wait, but could be a good check for the future! However, we should ensure that there is some way to ignore the output, because...

I think this is definitely still something that needs to be fixed, although depending on your confidence in understanding of the system you may want to start with a simpler...

I think the best places to start, issue-wise, may be #196 or #194. I'd also recommend just browsing the code and trying to understand how things work -- if you...

@gkapfham @corlettim Note that this PR's body does not have a valid versioning sentence. It should probably match the "minor" regex mentioned in the [README](https://github.com/Michionlion/pr-tag-release/blob/master/README.md), and so the word "small"...