Skip to content

Added/updated file headers#3019

Closed
cschuchardt88 wants to merge 0 commit intoneo-project:masterfrom
cschuchardt88:File-header
Closed

Added/updated file headers#3019
cschuchardt88 wants to merge 0 commit intoneo-project:masterfrom
cschuchardt88:File-header

Conversation

@cschuchardt88
Copy link
Member

@cschuchardt88 cschuchardt88 commented Dec 9, 2023 •
edited
Loading

Change Log

  • Added file header to .editorconfig
  • Made File header required
  • Updated all files headers
  • Ran code clean up

Code Clean up Fixes (Applied)

image

File Header will Now Read

// Copyright (C) 2015-2024 The Neo Project.
//
// RemoteNode.cs file belongs to the neo project and is free
// software distributed under the MIT software license, see the
// accompanying file LICENSE in the main directory of the
// repository or http://www.opensource.org/licenses/mit-license.php
// for more details.
//
// Redistribution and use in source and binary forms with or without
// modifications are permitted.

Note: Filename is automatically added. You just need to set or run code clean up in Visual Studio.
image

You can find code clean up in the menu on top.
image

@cschuchardt88 cschuchardt88 marked this pull request as ready for review December 9, 2023 16:02
shargon
shargon reviewed Dec 13, 2023
View reviewed changes
@cschuchardt88 cschuchardt88 mentioned this pull request Dec 14, 2023
Merged
@Jim8y
Copy link
Contributor

Jim8y commented Dec 14, 2023

you can just add that pr editconfig directly into this one, and update your pr. I can close my pr.

@cschuchardt88
Copy link
Member Author

cschuchardt88 commented Dec 14, 2023

it's up to you. Let me invite to my repo so you can edit if need be.

@cschuchardt88 cschuchardt88 marked this pull request as draft December 14, 2023 18:06
@cschuchardt88 cschuchardt88 deleted the File-header branch December 14, 2023 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shargon shargon shargon left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cschuchardt88 @Jim8y @shargon