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

[7.0] [IMP] update travis #110

Merged
merged 5 commits into from
May 25, 2015
Merged

[7.0] [IMP] update travis #110

merged 5 commits into from
May 25, 2015

Conversation

eLBati
Copy link
Member

@eLBati eLBati commented May 22, 2015

TODO: 8.0

@coveralls
Copy link

Coverage Status

Coverage remained the same at 45.93% when pulling fc4a2f7 on eLBati:update_travis into e030676 on OCA:7.0.

@archetipo
Copy link
Member

👍

1 similar comment
@tafaRU
Copy link
Member

tafaRU commented May 22, 2015

👍

- "2.7"
env:
global:
- VERSION="7.0" TESTS="0" LINT_CHECK="0"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here https://github.com/OCA/maintainer-quality-tools/blob/master/sample_files/.travis.yml#L14 I see also TRANSIFEX="0". Is there any reason to have removed it?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thus line 18 should be commented right ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On 22 May 2015 at 12:53, Yannick Vaucher notifications@github.com wrote:

Thus line 18 should be commented right ?

Right. Thanks. Done

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eLBati @yvaucher that row seems not invalidate the travis build, see https://travis-ci.org/OCA/l10n-italy/jobs/63628042 as reference.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Putting TRANSIFEX="0" shouldn't hurt

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As for the encrypted password for l10n-italy, see: OCA/maintainer-quality-tools#194

@tafaRU
Copy link
Member

tafaRU commented May 22, 2015

@eLBati about TODO see #111

@coveralls
Copy link

Coverage Status

Coverage remained the same at 45.93% when pulling 331ed15 on eLBati:update_travis into e030676 on OCA:7.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 45.93% when pulling 331ed15 on eLBati:update_travis into e030676 on OCA:7.0.

@tafaRU
Copy link
Member

tafaRU commented May 22, 2015

@eLBati please have a look at eLBati#57

@eLBati
Copy link
Member Author

eLBati commented May 24, 2015

Merged, thanks
On 22 May 2015 23:02, "Alex Comba" notifications@github.com wrote:

@eLBati https://github.com/eLBati please have a look at eLBati#57
eLBati#57


Reply to this email directly or view it on GitHub
#110 (comment).

@coveralls
Copy link

Coverage Status

Coverage remained the same at 45.93% when pulling bfe0c9f on eLBati:update_travis into e030676 on OCA:7.0.

@bwrsandman
Copy link

👍

tafaRU added a commit that referenced this pull request May 25, 2015
@tafaRU tafaRU merged commit 79a2971 into OCA:7.0 May 25, 2015
patrickt-oforce pushed a commit to patrickt-oforce/l10n-italy that referenced this pull request Mar 9, 2022
…balance_order

Revert "[FIX][l10n_it_account_balance_report]"
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.

None yet

6 participants