Skip to content
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

more detail required #12

Open
aspiers opened this issue Apr 26, 2013 · 6 comments
Open

more detail required #12

aspiers opened this issue Apr 26, 2013 · 6 comments

Comments

@aspiers
Copy link

aspiers commented Apr 26, 2013

[trello]
- 3 open assigned cards
- 2 cards in "Done" column

[twitter]
- 1 tweet

[bugzilla: bugzilla.novell.com]
- 4 open bugs
- 3 modified bugs

Would be much nicer if it reported the names of the cards, the content of the tweet, and the titles of the bugs.

@saschpe
Copy link
Owner

saschpe commented Apr 29, 2013

Sure, I wanted to expand the bugzilla one. Trello is harder since the metadata you get from the APIs is utter crap. But I'll try to add detail about which columns the cards where moved from / to. Do you really want to see all tweets in detail? That can generate a lot of noise in the report and I think one would want to mostly report the numbers.

@aspiers
Copy link
Author

aspiers commented Apr 29, 2013

How much do you tweet about work?! :) For me it's like once a month or something ...

@saschpe
Copy link
Owner

saschpe commented Apr 29, 2013

I try to have it as generic as possible, when you have more than 3 tweets, it's not useful to have that in the report imo.

@aspiers
Copy link
Author

aspiers commented Apr 29, 2013

For me it would still be useful because I would read all the tweet lines generated by rapport, and then manually replace them with a quick summary if there were too many to include all of them.

@saschpe
Copy link
Owner

saschpe commented Apr 29, 2013

Well, you can have a custom flavor by copying the twitter default template to ~/.rapport/templates and play around.

@aspiers
Copy link
Author

aspiers commented May 2, 2013

I've done this for the github plugin, and have vastly revamped it. Original output:

[github]
- 13 issues commented
- 4 create events:
  + 1 branch in SUSE-Cloud/barclamp-provisioner
  + 1 branch in aspiers/crowbar
  + 2 branchs in aspiers/rapport
- 1 gist
- 10 opened issues
- 4 forked repositories:
  + aspiers/jinja2 from mitsuhiko/jinja2
  + aspiers/rapport from saschpe/rapport
  + aspiers/barclamp-rabbitmq from crowbar/barclamp-rabbitmq
  + aspiers/barclamp-loadbalancer from crowbar/barclamp-loadbalancer
- 4 opened
 pull requests
- 15 push events summing up to:
  + 8 commits to SUSE-Cloud/automation
  + 1 commit to SUSE-Cloud/barclamp-crowbar
  + 1 commit to SUSE-Cloud/barclamp-provisioner
  + 4 commits to aspiers/crowbar
  + 10 commits to aspiers/git-config
  + 1 commit to aspiers/jinja2
  + 1 commit to aspiers/rapport

but merely showing counts of issues / pull requests filed, commits pushed, comments made etc. is not much use unless you want to go the "I made 5 more commits than you this week so I must be working harder/better" route which is clearly totally braindead ;) "8 commits to SUSE-Cloud/automation" doesn't really tell anybody anything.

New output is much more oriented around the content of the work. This might be too much detail but at least then it's easy to manually condense into a summary of the technical topics:

[github]

  - crowbar/crowbar
    + opened pull request #1792 (Pebbles install chef suse)
    + created comment on issue #1793 (Some additional pieces to bring install-chef-suse.sh back into 
shape)

  - SUSE-Cloud/barclamp-crowbar
    + pushed to refs/notes/upstreaming: 6b2be94d Notes added by 'git notes add'

  - aspiers/crowbar
    + created branch 'pebbles-install-chef-suse'
    + pushed to pebbles-install-chef-suse: 5a004f90 Add more stringent checks that /srv/tftpboot/repo
s are set up correctly (bnc#784494)
    + pushed 3 commits to pebbles-install-chef-suse
      - 876d0426 Check for root password to avoid accidental lock-out from admin node (bnc#782525)
      - b8dd32f9 Remove obsolete CROWBAR_TESTING mode
      - e4b2ccf6 Add more stringent checks that /srv/tftpboot/repos are set up correctly (bnc#784494)

  - aspiers/jinja2
    + pushed to master: 08f38a87 Prevent line statements eating newlines (fixes #52)

  - aspiers/rapport
    + created branch 'misc'
    + created branch 'xdg-compliance'
    + pushed to xdg-compliance: a1944cb6 comply with XDG Base Directory Specification (fixes #13)

  - SUSE-Cloud/barclamp-provisioner
    + created branch 'upstreaming'
    + pushed to refs/notes/upstreaming: d3c227df Notes added by 'git notes edit'
    + deleted branch 'upstreaming'

  - SUSE-Cloud/automation
    + pushed 2 commits to master
      - d31cffc3 change default local branch to be the 'suse' remote
      - 36027b6c update comment
    + pushed to master: 51e107fe compare-crowbar-upstream.sh: revamp to use git-icing
    + pushed 5 commits to master
      - 695ae4de fix check for top of crowbar tree
      - 30e22e13 handle missing barclamps more gracefully
      - 96e43a43 update hardcoded barclamp list
      - 4e622759 toplevel only needs to be set once
      - 564379c1 remove hardcoded barclamp list

  - crowbar/barclamp-rabbitmq
    + forked repo to aspiers/barclamp-rabbitmq

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

No branches or pull requests

2 participants