compiler-research / CppInterOp

A Clang-based C++ Interoperability Library
Other
43 stars 21 forks source link

Implemented GetFunctionArgName Module #49

Closed Smit1603 closed 1 year ago

Smit1603 commented 1 year ago
vgvassilev commented 1 year ago

Can you remove the merge commits?

vgvassilev commented 1 year ago

Can you mention in the commit message the related issue you are fixing?

Smit1603 commented 1 year ago
Smit1603 commented 1 year ago
vgvassilev commented 1 year ago

I suspect your main branch is out of sync. I'd propose doing something like this:

git checkout main
git remote add upstream https://github.com/compiler-research/InterOp.git
git fetch upstream
git reset --hard upstream/main

Then you can do:

git checkout -b my_new_branch_name
# make changes, then again
git fetch upstream
git rebase upstream/main
git push origin my_new_branch_name
vgvassilev commented 1 year ago

Btw, can you move the "fixes X part" two lines below the title of the commit?

vgvassilev commented 1 year ago
  • Are there any other issues/tasks which I can take upon and start for better learning purpose

I believe moving forward to the next open issue would be probably the best way to learn new things. I would suggest trying to set up locally cppyy as written in the readme here: https://github.com/compiler-research/cppyy-backend

Smit1603 commented 1 year ago
Smit1603 commented 1 year ago
  • Are there any other issues/tasks which I can take upon and start for better learning purpose

I believe moving forward to the next open issue would be probably the best way to learn new things. I would suggest trying to set up locally cppyy as written in the readme here: https://github.com/compiler-research/cppyy-backend

Ok sir , will start setting up cppyy locally, and will inform you whenever completed for further tasks/issues