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

Add Ghidra docs. #183

Closed
wants to merge 5 commits into from
Closed

Conversation

Hornos3
Copy link
Contributor

@Hornos3 Hornos3 commented Aug 26, 2024

Added Ghidra docs parsed through pandoc. It may have improper parses in some html files that is needed to be optimized soon.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Hornos3 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@tttturtle-russ
Copy link
Member

Your commit contains an unnecessary file .idea/.gitignore. plz remove it.

@mudongliang
Copy link
Member

  1. You need to sign CLA;
  2. You need to make CI happy if CI is correct;
  3. I suggest you could firstly collect the core articles into our repo;

@Hornos3
Copy link
Contributor Author

Hornos3 commented Aug 31, 2024

  1. You need to sign CLA;
  2. You need to make CI happy if CI is correct;
  3. I suggest you could firstly collect the core articles into our repo;

The CI seems to fail in citing pics, I've observed some md files merged and see that pics in the same directory of md file can be cited, but mine has a "image/" and it cannot recognize it. If it's the problem of astro, plz tell me and I'll adjust the file structure and make pics and mds in the same directory.

eg.
-- example.md (cited images/somepic.png)
-- images
|
-- somepic.png
The astro will report an error in such an example above

@Hornos3
Copy link
Contributor Author

Hornos3 commented Aug 31, 2024

@mudongliang

@mudongliang
Copy link
Member

按照之前仓库的情况,相对地址的picture应该是没问题的,你看看是不是用了绝对地址?另外,建议你先只放少数核心文件(10-20篇左右)

@Hornos3
Copy link
Contributor Author

Hornos3 commented Aug 31, 2024

@mudongliang 我看了那个报错,是相对的地址,不知道为什么就是不行

@Hornos3
Copy link
Contributor Author

Hornos3 commented Aug 31, 2024

I've mistaken the global email address, so the licence check is always pending. I'll close this and create another one.

@Hornos3 Hornos3 closed this Aug 31, 2024
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