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

Implement process.name #332

Merged
merged 8 commits into from
Oct 28, 2023
Merged

Implement process.name #332

merged 8 commits into from
Oct 28, 2023

Conversation

adm1nPanda
Copy link
Collaborator

modified:   docs/_docs/user-guide/eldritch.md
modified:   implants/lib/eldritch/src/process/name_impl.rs

What type of PR is this?

/kind feature
/kind eldritch-function

What this PR does / why we need it:

Implement process.name as per #58

Which issue(s) this PR fixes:

Fixes #58

	modified:   docs/_docs/user-guide/eldritch.md
	modified:   implants/lib/eldritch/src/process/name_impl.rs
Copy link
Collaborator

@hulto hulto left a comment

Choose a reason for hiding this comment

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

Hey adm1nPanda,
Thank you for submitting a PR!
We're excited to have you contribute I've left a few questions and suggestions. Really great job on your first PR 🙂

implants/lib/eldritch/src/process/name_impl.rs Outdated Show resolved Hide resolved
implants/lib/eldritch/src/process/name_impl.rs Outdated Show resolved Hide resolved
implants/lib/eldritch/src/process/name_impl.rs Outdated Show resolved Hide resolved
implants/lib/eldritch/src/process/name_impl.rs Outdated Show resolved Hide resolved
@adm1nPanda adm1nPanda requested a review from hulto October 24, 2023 01:55
Copy link
Collaborator

@hulto hulto left a comment

Choose a reason for hiding this comment

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

Looks great! 🚀

@adm1nPanda
Copy link
Collaborator Author

windows workflow failing in actions but passes in my lab. made some changes which i hope will clear this time

@adm1nPanda adm1nPanda requested a review from hulto October 24, 2023 15:46
@codecov
Copy link

codecov bot commented Oct 25, 2023

Codecov Report

Merging #332 (af0bbdf) into main (3c33ce0) will increase coverage by 0.11%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main     #332      +/-   ##
==========================================
+ Coverage   75.06%   75.17%   +0.11%     
==========================================
  Files         100      100              
  Lines        6545     6591      +46     
==========================================
+ Hits         4913     4955      +42     
- Misses       1545     1549       +4     
  Partials       87       87              
Files Coverage Δ
implants/lib/eldritch/src/process/name_impl.rs 87.50% <87.50%> (+87.50%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@hulto
Copy link
Collaborator

hulto commented Oct 26, 2023

Looks good! Go ahead and sqaush and merge.

@hulto hulto merged commit 8ccfa21 into spellshift:main Oct 28, 2023
8 checks passed
KCarretto pushed a commit that referenced this pull request Feb 1, 2024
 
Implement process.name (#332)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement process.name
4 participants