-
Notifications
You must be signed in to change notification settings - Fork 29
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
Bug: Job parallelism property is not displayed #141
Labels
bug
Something isn't working
Comments
I’ve created a PR to fix it but I can’t push it because I haven’t access right. Could you grant me up please ? |
Hey @tapesec You should be able to open a PR from a fork without any special access. Could you link us to your branch on GitHub? |
Oh sorry my bad I cloned the repo I didn’t fork it :( |
tapesec
added a commit
to tapesec/circleci-config-sdk-ts
that referenced
this issue
Sep 13, 2022
tapesec
added a commit
to tapesec/circleci-config-sdk-ts
that referenced
this issue
Sep 13, 2022
Closed in #139 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there an existing issue for this?
Current behavior
The parallelism option is supposed to available for a job.
This parallelism property is available into the Job class constructor but it is not passed to the generateContents method so you can’t see it in the final generated .yml config.
Minimum reproduction code
<script src="https://gist.github.com/tapesec/a2ba9bce9b2ca88542bd4cc05edca154.js"></script>
Steps to reproduce
No response
Expected behavior
The parallelism property passed to the Job class constructor is not returned from the generateContents method.
I added it.
CircleCI Config SDK version
v0.9.0-alpha.21
Node.js version
16.13.0
In which operating systems have you tested?
Other
No response
The text was updated successfully, but these errors were encountered: