Add Depth First Search in C

[2Clutch]

I Am Adding a New Code Snippet in an Existing Language

• I fixed Add Depth First Search in C #3296
• I named the pull request using Add {PROJECT} in {LANGUAGE} format

[github-advanced-security]

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

[rzuckerm]

I think you're going to have to add -lm to the build line in testinfo.yml in order to pull in the math libraries.

[2Clutch]

I think you're going to have to add -lm to the build line in testinfo.yml in order to pull in the math libraries.

Will keep this in mind moving forward. I ended up doing without it altogether.

[rzuckerm]

Recommended workflow:

• Keep your main branch clean by only using it to resync to the main branch of this repository
• Create a different branch for each sample program based on your main branch. For example, I would call the branch for this sample something like c-depth-first-search
• Keep pull requests in draft mode until they pass all the tests. That way I know when you're actually ready for me or Jeremy to review it

[2Clutch]

Recommended workflow:

• Keep your main branch clean by only using it to resync to the main branch of this repository
• Create a different branch for each sample program based on your main branch. For example, I would call the branch for this sample something like c-depth-first-search
• Keep pull requests in draft mode until they pass all the tests. That way I know when you're actually ready for me or Jeremy to review it

Noted. I don't think I've added or removed anything from draft (honestly didn't realize this was a feature). I just open a PR from the web and wait for review.

I've always waited for a PR to be approved before submitting the next one. Afaik, I don't have multiple PRs open, although #3299 is still open even though the relevant PR has been merged.

[rzuckerm]

Please see my previous comment about how to fix sqrt

[2Clutch]

I ended up implementing a solution without sqrt that successfully passes all the checks so I don't think an update is needed?

[rzuckerm]

It would be a good idea to update the testinfo.yml so that other samples can use sqrt and other math functions from math.h.

[rzuckerm]

Afaik, I don't have multiple PRs open, although #3299 is still open even though the relevant PR has been merged.

The "I fixed" field has two # characters. That's why it didn't auto-close. I just closed #3299

[rzuckerm]

Thanks for you contribution, and sorry for all the difficulty @2Clutch