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 input chor add Darma as participant and Darma Al Gani 's Portofolio into README #84

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Daarma-IC
Copy link
Collaborator

No description provided.

@fakhrip
Copy link
Member

fakhrip commented Aug 19, 2024

Sorry i wont allow this, please do the fix in the same PR -> #63
please close this and reopen that PR instead, if you are questioning of why im being so strict about this is because this is just how the way things work, if you cant follow this then youre going to have a pretty hard days going forward (in case you are pursuing software careers) 😄

so please do this just like how other academician do it, i know you can do this, please dont surrender and keep trying!

And just to be clear, you cant just do all changes in one go, follow what others have done, you should create the PR for adding your name first which should be in #63, and then later create new PR to add your website link

@Daarma-IC
Copy link
Collaborator Author

Daarma-IC commented Aug 20, 2024

Sorry about my foul before,
I want to do these, but unfortunately, i do deleted that forking before, and doing new forking for another.
Sorry about that mistake

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.

2 participants