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 static-program-analysis.md #230

Merged
merged 2 commits into from
Jul 30, 2024
Merged

Conversation

bartvanderwal
Copy link
Contributor

Alias 'linting' voor 'static code analysis' toegevoegd, en ook aantal concrete tools genoemd zoals SonarQube, Jenkins etc. zodat onderzoekers/developers/studenten anders mogelijk sneller herkennen dat zij dit ook al doen. Tevens ingredient toegevoegd om wel kritisch om te gaan, en niet te denken dat static code analysis een panacea is, zoals veel beginners wel doen.

Een en ander nog wat ingekort via raadplegen ChatGPT; full disclosure hier chat/bron: https://chat.openai.com/share/e0098fa9-e4e2-40f9-a7c3-a37f5c0ac9bb

Description

Closes #

[Please include a summary of the change and which issue is fixed]

Type of change

Please select the option that is relevant

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist

  • I have commented my code, particularly in hard-to-understand areas.
  • All the required documentation is added.
  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.

Additional information

[Provided extra information if necessary]

Alias 'linting' voor 'static code analysis' toegevoegd, en ook aantal concrete tools genoemd zoals SonarQube, Jenkins etc. zodat onderzoekers/developers/studenten anders mogelijk sneller herkennen dat zij dit ook al doen. Tevens ingredient toegevoegd om wel kritisch om te gaan, en niet te denken dat static code analysis een panacea is, zoals veel beginners wel doen.

Een en ander nog wat ingekort via raadplegen ChatGPT; full disclosure hier chat/bron:
https://chat.openai.com/share/e0098fa9-e4e2-40f9-a7c3-a37f5c0ac9bb
Copy link

netlify bot commented Apr 15, 2024

Deploy Preview for stellular-banoffee-031bf4 ready!

Name Link
🔨 Latest commit 60db4af
🔍 Latest deploy log https://app.netlify.com/sites/stellular-banoffee-031bf4/deploys/66a9378f05d49c00086454f9
😎 Deploy Preview https://deploy-preview-230--stellular-banoffee-031bf4.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.

Copy link

netlify bot commented Apr 15, 2024

Deploy Preview for test-ictresearchmethods ready!

Name Link
🔨 Latest commit 60db4af
🔍 Latest deploy log https://app.netlify.com/sites/test-ictresearchmethods/deploys/66a9378f7d582c0008e77356
😎 Deploy Preview https://deploy-preview-230--test-ictresearchmethods.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.

Copy link
Collaborator

@vkreuzen vkreuzen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inhoudelijk goedgekeurd, maar het is een array en mist nu separating commas.

Added ',' to the ingredients array
@jochemvogel jochemvogel merged commit d2952e1 into HBO-i:master Jul 30, 2024
14 checks passed
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.

3 participants