Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Updated contributing.md #155

Merged
merged 5 commits into from May 6, 2019
Merged

Updated contributing.md #155

merged 5 commits into from May 6, 2019

Conversation

abhinavsagar
Copy link
Contributor

No description provided.

@orcmid
Copy link

orcmid commented May 3, 2019

The update to contributing.md is useful, and it would be valuable to tie in howto's on the matter, such as what it takes to test and/or develop the code. Is there something easy to add to the list, such as

acquiring and operating with the code

that could also become an useful link to how-to information.

@abhinavsagar
Copy link
Contributor Author

@orcmid @mrkmarron Is it better now?

@orcmid
Copy link

orcmid commented May 5, 2019

Nice improvement, separate from what I was suggesting

@orcmid
Copy link

orcmid commented May 5, 2019

What's wrong with a leading topic,

# acquiring and operating with the code

so that folks who have never worked on a programming language or its processor can learn what the prerequisites are, and how simply to run the current build, become familiar with examples and tests?

I favor many of the expansions of this page, although it might be well to have separate pages with details and have short abstracts at this level?

I am sorry that I give advice but do not do the work. I am still learning how to synchronize my fork and its desktop clone, and then how to make a pull request.

Copy link
Contributor

@mrkmarron mrkmarron left a comment

Choose a reason for hiding this comment

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

Looks good with a few minor tweaks.

CONTRIBUTING.md Outdated
# Contributing
# Contributing to BosqueLanguage

We love your input! We want to make contributing to this project as easy as possible, whether it's:
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's delete "We love you input!" here.


# How Do I Submit A (Good) Bug Report?

Explain the problem and include additional details to help maintainers reproduce the problem:
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's include "Check if the same, or a similar, issue has already been reported" to minimize duplicates and help connect issues that may be related.

@abhinavsagar
Copy link
Contributor Author

@mrkmarron Done the required changes.

@mrkmarron mrkmarron merged commit 18f096f into microsoft:master May 6, 2019
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.

None yet

4 participants