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

Update appveyor.yml #1057

Merged
merged 8 commits into from Oct 10, 2019
Merged

Update appveyor.yml #1057

merged 8 commits into from Oct 10, 2019

Conversation

jmthomas
Copy link
Contributor

closes #1056

@codecov-io
Copy link

codecov-io commented Oct 10, 2019

Codecov Report

Merging #1057 into master will increase coverage by 0.26%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1057      +/-   ##
==========================================
+ Coverage   83.88%   84.14%   +0.26%     
==========================================
  Files         165      165              
  Lines       15739    15739              
==========================================
+ Hits        13203    13244      +41     
+ Misses       2536     2495      -41
Impacted Files Coverage Δ
lib/cosmos/packets/packet_item_limits.rb 96.61% <0%> (-3.39%) ⬇️
lib/cosmos/packets/structure_item.rb 92.5% <0%> (-1.88%) ⬇️
lib/cosmos/core_ext/array.rb 92.85% <0%> (-1.1%) ⬇️
lib/cosmos/tools/cmd_tlm_server/cmd_tlm_server.rb 92.3% <0%> (-0.3%) ⬇️
lib/cosmos/interfaces/tcpip_server_interface.rb 66.85% <0%> (-0.29%) ⬇️
lib/cosmos/config/config_parser.rb 92.55% <0%> (+0.46%) ⬆️
lib/cosmos/tools/launcher/launcher_config.rb 96.74% <0%> (+1.62%) ⬆️
lib/cosmos/gui/qt.rb 37.76% <0%> (+2.1%) ⬆️
lib/cosmos.rb 93.1% <0%> (+3.44%) ⬆️
lib/cosmos/io/serial_driver.rb 92.85% <0%> (+3.57%) ⬆️
... and 3 more

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 e56f74b...0a87f9a. Read the comment docs.

@jmthomas jmthomas requested a review from a user October 10, 2019 16:54
@@ -9,7 +9,6 @@ rvm:
- 2.6.3
- 2.5.6
- 2.4.6
- 2.3.8
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is officially deprecated by Ruby so while it doesn't mean we don't support it I don't want it to be part of the Travis builds

@@ -114,7 +114,7 @@ spec = Gem::Specification.new do |s|
s.add_development_dependency 'simplecov', '~> 0.15'
s.add_development_dependency 'codecov', '~> 0.1'
s.add_development_dependency 'benchmark-ips', '~> 2.0'
s.add_development_dependency 'ruby-prof', '~> 0.16' if RUBY_ENGINE == 'ruby' # MRI Only
s.add_development_dependency 'ruby-prof', '~> 1.0' if RUBY_ENGINE == 'ruby' # MRI Only
Copy link
Contributor Author

Choose a reason for hiding this comment

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

1.0 only supports Ruby 2.4 or higher


cache:
# If one of the files after the right arrow changes, cache will be invalidated
- vendor\bundle -> appveyor.yml, Gemfile, cosmos.gemspec
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hopefully this will make the appveyor builds faster

@ghost
Copy link

ghost commented Oct 10, 2019

👍

@jmthomas jmthomas merged commit e4d9f1e into master Oct 10, 2019
@jmthomas jmthomas deleted the fix_appveyor branch October 10, 2019 21:25
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.

AppVeyor broken
2 participants