Skip to content

Conversation

@Ekultek
Copy link
Contributor

  • Have to have the right version to create an issue
  • Changes the way that issue "identifiers" are created
  • Some minor fixes

khast3xand others added 3 commits April 14, 2019 22:25
… version it will not create an issue, also changes how the identifier is created
@Ekultek
Copy link
ContributorAuthor

Also the docker stuff

@NullArray
Copy link
Owner

Thought i merged the docker stuff already? Anyway i'll add my review.

Copy link
Owner

@NullArrayNullArray left a comment

Choose a reason for hiding this comment

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

Why are we getting rid of the README.md in the docker directory. I have referenced that in the main README, with regards to the docker compose stuff.

@NullArray
Copy link
Owner

If i am missing something, please let me know. It could be an oversight on my part but i'd like to make sure before i merge it into the master branch.

@EkultekEkultek requested a review from NullArrayApril 18, 2019 17:31
Copy link
Owner

@NullArrayNullArray left a comment

Choose a reason for hiding this comment

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

Thank you for addressing the issue. Approved and merged.

@NullArrayNullArray merged commit 94677e7 into masterApr 18, 2019
@Ekultek
Copy link
ContributorAuthor

<3

Sign up for freeto 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

@Ekultek@NullArray