Skip to content

#3671 Add attribute tags - #3702

Merged
memsharded merged 6 commits into
conan-io:developfrom
uilianries:feature/attribute-tags
Oct 24, 2018
Merged

memsharded merged 6 commits into
conan-io:developfrom
uilianries:feature/attribute-tags

Conversation

@uilianries

@uilianries uilianries commented Oct 9, 2018

Copy link
Copy Markdown
Member

Changelog: Feature: Added topics attribute to the ConanFile to specify topics (a.k.a tags, a.k.a keywords) to the recipe.

This PR is related to #3671

Also, I added author as default attribute when running conan new

Docs PR: conan-io/docs#875

  • Refer to the issue that supports this Pull Request.
  • If the issue has missing info, explain the purpose/use case/pain/need that covers this Pull Request.
  • I've read the Contributing guide.
  • I've followed the PEP8 style guides for Python code.
  • I've opened another PR in the Conan docs repo to the develop branch, documenting this one.

@ghost ghost added the contributor pr label Oct 9, 2018
@memsharded memsharded added this to the 1.9 milestone Oct 9, 2018
Comment thread conans/client/cmd/new.py Outdated
- Support new attribute on Conafile: tags

Signed-off-by: Uilian Ries <uilianries@gmail.com>
- Validate both inspect and info commands using tags
  as attribute

Signed-off-by: Uilian Ries <uilianries@gmail.com>
- Fix attribute order to validate inspect command

Signed-off-by: Uilian Ries <uilianries@gmail.com>
- We believe that topics is a better name, because tags remember svn
  and git features.
- We voted also for labels but topics sounds better.
- Thanks to @SSE4 for 'topics'

Signed-off-by: Uilian Ries <uilianries@gmail.com>
- Homepage is optional but will be listed by info and inspect
  commands
- Update unit tests to validate homepage
- Update topics to be used as tuple

Signed-off-by: Uilian Ries <uilianries@gmail.com>
@uilianries
uilianries force-pushed the feature/attribute-tags branch from 5a64a34 to 7a90b4f Compare October 22, 2018 14:18
@uilianries

Copy link
Copy Markdown
Member Author

@danimtb

danimtb commented Oct 23, 2018

Copy link
Copy Markdown
Member

Relaunched CI

@jgsogo

jgsogo commented Oct 23, 2018

Copy link
Copy Markdown
Contributor

This is adding topics and homepage, it should be mentioned in the changelog and maybe documented, shouldn't it?

@uilianries

Copy link
Copy Markdown
Member Author

There is a PR related to topics on conan-io/docs but I don't know how to proceed about changelog

@jgsogo

jgsogo commented Oct 24, 2018

Copy link
Copy Markdown
Contributor

We generate the changelog for a version automatically by parsing all the PRs that are merged into that version, our script takes the Changelog: (Feature|Fix|Bugfix): whatever line in the first comment.

@uilianries

Copy link
Copy Markdown
Member Author

nice 👍

Comment thread conans/test/command/info_test.py Outdated
Signed-off-by: Uilian Ries <uilianries@gmail.com>
@memsharded
memsharded merged commit fd456eb into conan-io:develop Oct 24, 2018
@ghost ghost removed the contributor pr label Oct 24, 2018
grisumbras pushed a commit to grisumbras/conan that referenced this pull request Dec 27, 2018
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.

4 participants