Skip to content

feat: add format_header#549

Open
infdahai wants to merge 4 commits into
cmu-db:masterfrom
infdahai:format_header
Open

feat: add format_header#549
infdahai wants to merge 4 commits into
cmu-db:masterfrom
infdahai:format_header

Conversation

@infdahai

@infdahai infdahai commented Mar 24, 2023

Copy link
Copy Markdown

this pr's target is to solve issue #281 .

@infdahai

Copy link
Copy Markdown
Author

and we should use this header template.

//===----------------------------------------------------------------------===//
//
//                         BusTub
//
// rid.h
//
// Identification: src/include/common/rid.h
//
// Copyright (c) 2015-2019, Carnegie Mellon University Database Group
//
//===----------------------------------------------------------------------===//

@skyzh

skyzh commented Mar 25, 2023

Copy link
Copy Markdown
Member

You don't need to replace all headers now. We will need to manually apply it to the private repo. As long as the script works, then I'm happy to get it merged.

@infdahai

infdahai commented Mar 25, 2023

Copy link
Copy Markdown
Author

I guess the format_header py can insert some place and then execute the commands like 'make project2'

@infdahai infdahai changed the title [feat] add format_header feat: add format_header Mar 26, 2023
infdahai added 4 commits April 3, 2023 01:13
1. still incomplete file.

Signed-off-by: clundro <clusternlh@gmail.com>
Signed-off-by: clundro <clusternlh@gmail.com>
Signed-off-by: clundro <clusternlh@gmail.com>
Signed-off-by: clundro <clusternlh@gmail.com>
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