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

21269 Provide tags in package Announcements-Core for better structure #823

Merged
merged 8 commits into from Feb 27, 2018

Conversation

Projects
None yet
3 participants
@astares

This comment has been minimized.

Member

astares commented Feb 10, 2018

Only tagging, no change in behavior

astares added some commits Feb 12, 2018

@Ducasse

This comment has been minimized.

Member

Ducasse commented Feb 16, 2018

Hi torsten
Please do not do empty commit because it will make our life more difficult when chasing parent commits.

@Ducasse

This comment has been minimized.

Member

Ducasse commented Feb 16, 2018

I'm not sure that it is worth to have 3 tags for 10 classes in total.

@astares

This comment has been minimized.

Member

astares commented Feb 16, 2018

Sure it is worth - because giving a package structure means to give also informations to the users.
This is independent from the number of contained classes.

astares added some commits Feb 23, 2018

Merge branch '21269-Provide-tags-in-package-Announcements-Core-for-be…
…tter-structure' of https://github.com/astares/pharo into 21269-Provide-tags-in-package-Announcements-Core-for-better-structure

@MarcusDenker MarcusDenker merged commit 3e3e1ea into pharo-project:development Feb 27, 2018

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment