Skip to content

Removed deadcode and updated makefile#279

Open
Vinuthna-G-1091 wants to merge 1 commit intocodenet:mainfrom
Vinuthna-G-1091:Reviewing_the_changes
Open

Removed deadcode and updated makefile#279
Vinuthna-G-1091 wants to merge 1 commit intocodenet:mainfrom
Vinuthna-G-1091:Reviewing_the_changes

Conversation

@Vinuthna-G-1091
Copy link
Copy Markdown

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
Copy link
Copy Markdown

Copilot AI left a comment

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.

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.

2 participants