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.

Sign up for GitHub

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

Jump to bottom

Configure pdd and est in travis and rultor #581

Closed
krzyk opened this issue Jan 1, 2016 · 9 comments
Closed

Configure pdd and est in travis and rultor #581

krzyk opened this issue Jan 1, 2016 · 9 comments
Labels
bug

Comments

@krzyk
Copy link
Collaborator

krzyk commented Jan 1, 2016

  1. Current travis build don't check if puzzles created are correct (using pdd tool), same for estimates (using est tool).
  2. Moreover rultor build has pdd checks, but doesn't have est checks, this also has to be fixed (by adding est --dir=est --file=/dev/null.
@mkordas
Copy link
Contributor

mkordas commented Jan 1, 2016

@krzyk please assign me when it will be valid bug

@krzyk
Copy link
Collaborator Author

krzyk commented Jan 3, 2016

@davvd valid bug

@davvd
Copy link

davvd commented Jan 4, 2016

@davvd valid bug

@krzyk I just added bug tag here

@davvd davvd added the bug label Jan 4, 2016
@krzyk
Copy link
Collaborator Author

krzyk commented Jan 4, 2016

@davvd assign @mkordas please

@davvd davvd added DEV labels Jan 4, 2016
@davvd
Copy link

davvd commented Jan 4, 2016

@davvd assign @mkordas please

@krzyk yep, done. @mkordas it is your task now

mkordas added a commit to mkordas/qulice that referenced this issue Jan 4, 2016
@mkordas
yegor256#581 Configure pdd and est in Travis and Rultor
1a7f513
mkordas added a commit to mkordas/qulice that referenced this issue Jan 4, 2016
@mkordas
yegor256#581 Configure pdd and est in Travis and Rultor
3421744
@mkordas mkordas mentioned this issue Jan 4, 2016
Merged
mkordas added a commit to mkordas/qulice that referenced this issue Jan 4, 2016
@mkordas
yegor256#581 Configure pdd and est in Travis and Rultor
3ebaf6e
mkordas added a commit to mkordas/qulice that referenced this issue Jan 4, 2016
@mkordas
yegor256#581 Change order of pdd and est
53c6f25
mkordas added a commit to mkordas/qulice that referenced this issue Jan 4, 2016
@mkordas
yegor256#581 Exclude PDD for ITs
6d562f0
mkordas added a commit to mkordas/qulice that referenced this issue Jan 4, 2016
@mkordas
yegor256#581 Configure pdd and est in Travis and Rultor
c759faf
mkordas added a commit to mkordas/qulice that referenced this issue Jan 4, 2016
@mkordas
yegor256#581 Change order of pdd and est
722a03a
mkordas added a commit to mkordas/qulice that referenced this issue Jan 4, 2016
@mkordas
yegor256#581 Exclude PDD for ITs
c1ac7d6
@davvd
Copy link

davvd commented Jan 5, 2016

@krzyk many thanks, 30 mins added to your acc for reporting this bug, pmt ID 73861370

mkordas added a commit to mkordas/qulice that referenced this issue Jan 6, 2016
@mkordas
yegor256#581 Add 'set +e' after each part of execution
b1d60d5
@mkordas
Copy link
Contributor

mkordas commented Jan 6, 2016

@krzyk merged in #588, can you close issue?

@krzyk
Copy link
Collaborator Author

krzyk commented Jan 6, 2016

@mkordas thanks

@krzyk krzyk closed this as completed Jan 6, 2016
@davvd
Copy link

davvd commented Jan 8, 2016

@mkordas paid, thanks, added 38 mins to your account, payment 568f9525d456112c5500016d, 67 hours and 26 mins was spent. m=4046, that's why a bonus here for fast delivery. added +38 to your rating, now it is equal to +1587

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@krzyk @mkordas @davvd