Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

2.0.0 #19

Merged
merged 1 commit into from
Dec 8, 2024
Merged

2.0.0 #19

merged 1 commit into from
Dec 8, 2024

Conversation

UlisesGascon
Copy link
Member

@UlisesGascon UlisesGascon commented Dec 8, 2024

What's Changed

⚠️ BREAKING CHANGES ⚠️

In the version 1.x we extracted the data from the spreadsheet and convert it a json file to generate the details and the implementation groups. Currently, since 2.x we moved away from this and we directly use the compliance_checks table from the dashboard.

We keep the same approach of collecting the data as a json and then generate all the files dynamically. All the process now is done with a GitHub action:

Update the compliance checks

  1. Go to Actions: Sync and update Compliance Checks and run the action manually from the main branch. how-to
  2. This will generate a PR with the title [AUTO] Sync with dashboard database and it will assign it to you (direct access). Please review the content and merge it when you feel ready.

Notable Changes

Data Source migration
Removed legacy script that converted html data into json (entities extraction). This include jsdom dependency (65bb52f), the script itself (208f848) and the npm command npm run extract-standards (5b8044e)

  • The population scripts populate-details (bec24b1) and populate-implementations (5703528) are now using the new data source (dashboard database dump)
  • The detatils files now include more information, also some minor bugs were solved (5550e40)

Add Manual Annotations to Dynamic Content

Added the ability to combine dynamic content with manual annotations in the details (09d66e1) and implementations (8f37303) files.

Dynamic content is managed by the @ulisesgascon/text-tags-manager package, a core dependency of this project (016063b). Tags within these files are automatically updated or inserted whenever a new detail or implementation is added (49ba99c).

For example, the githubOrgMFA implementation has been updated to include manual annotations (revert b216d3c), restoring annotations that were previously removed (8460de4).

To support this change, the documentation has been updated (cc83c54) with clear instructions on working with dynamic content and tags. The updated section is as follows:

Improve the content via PR(s)

If you want to enhance any page, you can do so as you would in any other project (via a Pull Request, example). However, please note certain rules, as some parts of the files are dynamically generated, and your changes could be overwritten.

Rules

  1. Metadata is added automatically. Manual additions or modifications to metadata are not allowed.
  2. You can contribute any content to any file, but avoid making changes within the sections enclosed by specific tags, as these sections are dynamically >generated. For example:
    OK
    <!-- DESCRIPTION:START -->
    AVOID (AUTOMATED)
    <!-- DESCRIPTION:END -->
    OK
    

Other changes

  • Add evaluation criteria for githubOrgMFA compliance check
  • Improve documentation and update the content with all the changes

PRs

New Contributors

Changelog

Full Changelog: 1.0.0...main

@UlisesGascon UlisesGascon self-assigned this Dec 8, 2024
@UlisesGascon UlisesGascon added this to the Use dashboard database milestone Dec 8, 2024
Copy link

netlify bot commented Dec 8, 2024

Deploy Preview for openjs-security-program-standards ready!

Name Link
🔨 Latest commit 7e17aea
🔍 Latest deploy log https://app.netlify.com/sites/openjs-security-program-standards/deploys/6755f345c32a1d0008b273d7
😎 Deploy Preview https://deploy-preview-19--openjs-security-program-standards.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@OpenPathfinder OpenPathfinder locked as resolved and limited conversation to collaborators Dec 8, 2024
@UlisesGascon UlisesGascon marked this pull request as ready for review December 8, 2024 19:33
@UlisesGascon UlisesGascon merged commit 5d511c3 into main Dec 8, 2024
5 checks passed
@UlisesGascon UlisesGascon deleted the release/2.0.0 branch December 8, 2024 19:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant