-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update Readme #261
Update Readme #261
Conversation
README.md
Outdated
@@ -1,55 +1,31 @@ | |||
[](https://www.gnu.org/licenses/lgpl-3.0) | |||
# BHoM | |||
[](https://ci.appveyor.com/api/projects/status/bhom_ui/branch/master) | |||
[](https://dev.azure.com/BHoMBot/BHoM/_build/latest?definitionId=17&branchName=master) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the definitionId for BHoM_UI ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
19
- full badge is: [](https://dev.azure.com/BHoMBot/BHoM/_build/latest?definitionId=19&branchName=master)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To avoid asking the same question for each PR, how can I find the number for each repo ?
I have fixed the definitionId so it should be ready for review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM @adecler - in line with BHoM_Adapter so am happy 😄
/azp run BHoM_UI.CheckInstaller |
Azure Pipelines successfully started running 1 pipeline(s). |
Issues addressed by this PR
Closes #260
Updating the Readme following the changes already made on BHoM adapter