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

set env in ProcessInvoker sanitized #2280

Merged
merged 7 commits into from
Dec 19, 2022
Merged

set env in ProcessInvoker sanitized #2280

merged 7 commits into from
Dec 19, 2022

Conversation

ruvceskistefan
Copy link
Contributor

No description provided.

@ruvceskistefan ruvceskistefan requested a review from a team as a code owner November 24, 2022 07:59
@pje
Copy link
Member

pje commented Dec 14, 2022

Thanks for the contribution, @ruvceskistefan!

Can you add a small test for the new behavior?

https://github.com/actions/runner/blob/main/src/Test/L0/ProcessInvokerL0.cs

@ruvceskistefan
Copy link
Contributor Author

@pje tests added

Copy link
Contributor

@JoannaaKL JoannaaKL left a comment

Choose a reason for hiding this comment

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

looks good

@JoannaaKL JoannaaKL merged commit f1b1532 into actions:main Dec 19, 2022
fhammerl pushed a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized
fhammerl pushed a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized
fhammerl pushed a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized
fhammerl pushed a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized
fhammerl pushed a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized
fhammerl added a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized (#2280)

* set env in ProcessInvoker sanitized

* Update runnerversion and rel notes

---------

Co-authored-by: Stefan Ruvceski <96768603+ruvceskistefan@users.noreply.github.com>
fhammerl added a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized (#2280)

* set env in ProcessInvoker sanitized

* Update runnerversion and rel notes

---------

Co-authored-by: Stefan Ruvceski <96768603+ruvceskistefan@users.noreply.github.com>
fhammerl added a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized (#2280)

* set env in ProcessInvoker sanitized

* Update release notes and runnerversion

---------

Co-authored-by: Stefan Ruvceski <96768603+ruvceskistefan@users.noreply.github.com>
fhammerl added a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized (#2280)

* set env in ProcessInvoker sanitized

* Update release notes and runnerversion

* Update runnerversion

---------

Co-authored-by: Stefan Ruvceski <96768603+ruvceskistefan@users.noreply.github.com>
fhammerl added a commit that referenced this pull request Jan 30, 2023
* set env in ProcessInvoker sanitized (#2280)

* set env in ProcessInvoker sanitized

* Update runnerversion and rel notes and make it prerelase

---------

Co-authored-by: Stefan Ruvceski <96768603+ruvceskistefan@users.noreply.github.com>
nikola-jokic pushed a commit to nikola-jokic/runner that referenced this pull request May 12, 2023
* set env in ProcessInvoker sanitized
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.

4 participants