-
Notifications
You must be signed in to change notification settings - Fork 11
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 structure and presentation #40
Update structure and presentation #40
Conversation
1e9fdd2
to
3698507
Compare
Make `README.md` instructions compatible with other repositories. Signed-off-by: Razvan Deaconescu <[email protected]>
Remove trailing whitespace in `COPYING.md`. Signed-off-by: Razvan Deaconescu <[email protected]>
Sync `CONTRIBUTING.md` contents with the one in the template repository. Signed-off-by: Razvan Deaconescu <[email protected]>
Use correct links (URLs) to binary security content (SSS) in config.yaml file. Signed-off-by: Razvan Deaconescu <[email protected]>
Generated content (mostly slides) was added by mistake in the repository. Remove it. Signed-off-by: Razvan Deaconescu <[email protected]>
Add `.gitignore` file to ignore generated slide output files. Namely the `_site/` directory and the `slides.md` file (resulted from preprocessing). Signed-off-by: Razvan Deaconescu <[email protected]>
Rename `activities/` directory to `drills/`. And rename `assets/` to `media/`. Also place `README.md` in `reading/` directory. Signed-off-by: Razvan Deaconescu <[email protected]>
3698507
to
cb587e7
Compare
I added the |
It looks ok to me, I think we should ignore for now |
3a77ace
to
51965aa
Compare
Reformat content to make it publishable via Docusaurus. The `pwntools-intro` section was previously stored outside the `chapters/` directory. Move it inside the `chapters/` directory. Update formatting, links and configuration file (`config.yaml`) to new structure. Reformat Python code using `black`. Apply linting (via `markdownlint-cli`) to Markdown files. Signed-off-by: Razvan Deaconescu <[email protected]>
The `CPPLINT.cfg` file in the actions repository doesn't work. The configuration file has to be part of the root repository, so we add it here. Signed-off-by: Razvan Deaconescu <[email protected]>
51965aa
to
e809d63
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I scrolled through the content and couldn't find any rendering issues. It looks alright.
Prerequisite Checklist
Description of changes
Add entire contents of binary security class. Configure appropriate rendering in
config.yaml
. Make updates to content: placement, links, formatting.