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

Use common example rolling stock files in infra-generator #3193

Merged
merged 1 commit into from
Feb 17, 2023

Conversation

Castavo
Copy link
Contributor

@Castavo Castavo commented Feb 17, 2023

In the infra-generator we use a set of rolling stocks different from the rolling stocks in the core/example/rolling_stocks dir.

To avoid maintaining the two sets, we change the infra generator to use the rollings stocks in the dir mentioned above.

@Castavo Castavo requested a review from a team as a code owner February 17, 2023 09:33
@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Merging #3193 (feb4748) into dev (143409e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #3193   +/-   ##
=========================================
  Coverage     40.33%   40.33%           
  Complexity     1711     1711           
=========================================
  Files           630      630           
  Lines         18766    18766           
  Branches       2352     2352           
=========================================
  Hits           7570     7570           
  Misses        10782    10782           
  Partials        414      414           
Flag Coverage Δ
core 83.76% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@multun multun enabled auto-merge (rebase) February 17, 2023 09:51
@multun multun merged commit d060354 into dev Feb 17, 2023
@multun multun deleted the bpt/fix-infra-generator-rs branch February 17, 2023 09:53
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.

3 participants