Skip to content

Removed deadcode and updated makefile#279

Merged
sai-kaushik-s merged 1 commit into
codenet:mainfrom
Vinuthna-G-1091:Reviewing_the_changes
May 9, 2026
Merged

Removed deadcode and updated makefile#279
sai-kaushik-s merged 1 commit into
codenet:mainfrom
Vinuthna-G-1091:Reviewing_the_changes

Conversation

@Vinuthna-G-1091

Copy link
Copy Markdown
Contributor

fix: update Makefile cleanup and remove obsolete 02.out

  • Removed non-executable 02.out file
  • Added 02.main to make clean target

Improved cleanup logic in Makefile

  • Added rm -f in the clean target to remove generated files
  • This avoids errors if files do not exist
  • Ensures consistent cleanup behavior across builds

Copilot AI review requested due to automatic review settings March 21, 2026 03:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR improves build artifact cleanup for exercise 02 by removing an obsolete output file and making the make clean target more robust.

Changes:

  • Remove the obsolete 02/02.out file from the repository.
  • Update 02/Makefile’s clean target to use rm -f and to remove the 02.main binary.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
02/Makefile Makes clean idempotent (rm -f) and ensures the linked binary (02.main) is removed.
02/02.out Deleted obsolete/non-executable output artifact.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sai-kaushik-s sai-kaushik-s merged commit 8c25c22 into codenet:main May 9, 2026
3 of 4 checks passed
@sai-kaushik-s sai-kaushik-s added Spring2026 PR made be Spring 2026 students, possibly for errata participation. Remove Dead Code Removes some dead code from code base labels May 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Remove Dead Code Removes some dead code from code base Spring2026 PR made be Spring 2026 students, possibly for errata participation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants