Skip to content

Update issue_template#223

Merged
SD10 merged 5 commits into
MessageKit:v0.9.0from
VictorAlagwu:master
Oct 4, 2017
Merged

Update issue_template#223
SD10 merged 5 commits into
MessageKit:v0.9.0from
VictorAlagwu:master

Conversation

@VictorAlagwu

Copy link
Copy Markdown
Member

No description provided.

Comment thread pull-request-template.md Outdated

**Toolchain Version:** …

**SDK Version:** …

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we change this to "MessageKit Version:" ?

Comment thread pull-request-template.md Outdated

**Target Platform:** …

**Toolchain Version:** …

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"Swift Version:"

Comment thread pull-request-template.md Outdated

Where has this been tested?
---------------------------
**Operating System:** …

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"iOS Version:"

Comment thread pull-request-template.md Outdated

**Platform:** …

**Target Platform:** …

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"Devices/Simulators:"

Comment thread pull-request-template.md Outdated
---------------------------
**Operating System:** …

**Platform:** …

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Don't need this one

@SD10 SD10 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@VictorAlagwu Good work. Just a few changes to make it more specific to our project. No need for a new PR. Just make the changes with another commit.

@VictorAlagwu

Copy link
Copy Markdown
Member Author

I have done that

@Sherlouk Sherlouk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we also move these to the .github directory as to not clog up the root directory?

Also I have mild OCPD so can we name the files consistently: issue_template.md (same) and pull_request_template.md (underscores, not hypens).

@VictorAlagwu

Copy link
Copy Markdown
Member Author

Will do that

@VictorAlagwu

VictorAlagwu commented Oct 4, 2017

Copy link
Copy Markdown
Member Author

Done

@SD10

SD10 commented Oct 4, 2017

Copy link
Copy Markdown
Member

Thank you @VictorAlagwu 🎉 👍

@SD10 SD10 changed the base branch from master to v0.9.0 October 4, 2017 10:39
@SD10 SD10 merged commit 42769ea into MessageKit:v0.9.0 Oct 4, 2017
@VictorAlagwu

Copy link
Copy Markdown
Member Author

You are welcome

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