Skip to content
This repository was archived by the owner on Jan 10, 2023. It is now read-only.

Updated appveyor enabled values detection #114

Closed
wants to merge 1 commit into from

Conversation

andrew-aladev
Copy link

Hello, please update with the recent changes of appveyor environment.

APPVEYOR - True (true on Ubuntu image) if build runs in AppVeyor environment
CI - True (true on Ubuntu image) if build runs in AppVeyor environment

@codecov
Copy link

codecov bot commented Dec 6, 2020

Codecov Report

Merging #114 (0dbf560) into master (f677699) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
+ Coverage   90.52%   90.55%   +0.02%     
==========================================
  Files           1        1              
  Lines         359      360       +1     
==========================================
+ Hits          325      326       +1     
  Misses         34       34              
Impacted Files Coverage Δ
lib/codecov.rb 90.55% <100.00%> (+0.02%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f677699...0dbf560. Read the comment docs.

@andrew-aladev
Copy link
Author

Workaround:

environment:
  matrix:
    - job_name: Ubuntu
      appveyor_build_worker_image: Ubuntu2004
      APPVEYOR: True
      CI: True

Comment on lines +50 to +51
ci = if (ENV['CI'] == 'True' && ENV['APPVEYOR'] == 'True') ||
(ENV['CI'] == 'true' && ENV['APPVEYOR'] == 'true')
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we just do ENV['CI'].downcase and ENV['APPVEYOR'].downcase?

assert_equal('build', result['params'][:build])
assert_equal('owner/repo/job', result['params'][:job])
assert_equal('f881216b-b5c0-4eb1-8f21-b51887d1d506', result['params']['token'])
%w[True true].each do |enabled_value|
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd rather this be two different tests

@thomasrockhu-codecov
Copy link
Contributor

Closing due to deprecation

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants