Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update sanity check to cover .github and .vscode folders #3557

Merged
merged 3 commits into from
Aug 5, 2022

Conversation

reyang
Copy link
Member

@reyang reyang commented Aug 5, 2022

@reyang reyang marked this pull request as ready for review August 5, 2022 20:24
@reyang reyang requested a review from a team August 5, 2022 20:24
@reyang reyang added the infra Infra work - CI/CD, code coverage, linters label Aug 5, 2022
Comment on lines -52 to -53
# ℹ️ Command-line programs to run using the OS shell.
# 📚 https://git.io/JvXDl
Copy link
Member

Choose a reason for hiding this comment

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

but but... I like emojis. Can we have sanitycheck require minimum one emoji per file?

Copy link
Member Author

@reyang reyang Aug 5, 2022

Choose a reason for hiding this comment

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

Is there an emoji for OpenTelemetry (maybe 🔭📺📊📈📉)? Might be a good thing (although it's not friendly to developers who use Notepad.exe or vi via telnet) 😃

Copy link
Member

Choose a reason for hiding this comment

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

(although it's not friendly to developers who use Notepad.exe or vi via telnet)

I have a solution:

MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWX0KNWMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWX0OkkOKNWMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWX0OkkkkkkOKNWMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMNKOkkkkkkkkkkOKNWMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMN0kkkkkkkkkkkkkOKNWMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWNKOkkkkkkkkkkkkkOKNWMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWNKOkkkkkkkkkkkkkOKNWMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWXOkOXWMWNKOkkkkkkkkkkkkkOKNWMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWXkl:;:cxKWMWNKOkkkkkkkkkkkkkOKNWMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWWXkl:;;:;;:lxKWMWNKOkkkkkkkkkkkkkOKNWM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWXkdl:;:;;;;;;;:cxKWMWNKOkkkkkkkkkkkkkOXW
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWXkl:;;;;:;;;;;;;;::cxKWMWNKOkkkkkkkkkkkOKW
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWXkl:::;;;;::;;;:;;:;;;:cxKWMWNKOkkkkkkkO0XWM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMKo:;;;;:;;:;;;;;;;;;::;;;:cxKWMWNKOkkkO0XWMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMKo:;;;;;;;;;;;;;;;;:;;:;;:;:ckNMMWNKKKXWMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMNOl:;;::;;;;;;;;;;;;;;;;;;;:dXMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMNOo:::;;::ccllcc:;:;;:;:lkXWMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMWKkdx0NMMXOxooxO0XXNNXK0kdl::lkXWMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMW0dc:;;:d0NMMWWWWMMWWWWWMMMWXOOXWMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMW0dc;::;;;:cOWMMMMWXK00OO0KXNWMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMW0dc:;;;;;;;:oKMMMWX0kkkOOOOkkOKNMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMW0dc:;;;;;;;;;l0WMWNKOkk0XNWNNKOkk0NMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMW0dc;;;;;;;;;;;;oXMMNK0kk0NMMMMMWKkkOKWMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMW0dc;;;;;:;;;;;;;;oXMMNX0kk0NMMMMMWKkkOXWMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMXxc:;;:;;:;;;:;;;;;c0WMWWXOkkOKXNNX0OkkKNMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMM0l:;;;;;::;::;;;;;:ckNMMMWX0OkkkOOkkkOKNMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMWKdc:;;:;;;;;;;;;cd0NMMMMMMWNXK0000KXWWMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMWKdc:;;:;;:;:cd0NMMMMMMMMMMMMWWWWMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMWWXKXNWMWKdc:;;;;cd0NMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMWNK0kkkOKNWMWKxc:cd0NMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MNKKXNNX0kkkkkkk0NMMMWXKKWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
WXOkkOOkkkkkkkkOXWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MWX0OkkkkkkkkOKNWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMWX0OkkkkOKNWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMWX0OkkOKWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMWX0OkOXMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMWNXNWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM

Copy link
Member Author

Choose a reason for hiding this comment

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

Added more 💰💰💰:

 $$$$$$\                             $$$$$$$$\        $$\                                    $$\                         
$$  __$$\                            \__$$  __|       $$ |                                   $$ |                        
$$ /  $$ | $$$$$$\   $$$$$$\  $$$$$$$\  $$ | $$$$$$\  $$ | $$$$$$\  $$$$$$\$$$$\   $$$$$$\ $$$$$$\    $$$$$$\  $$\   $$\ 
$$ |  $$ |$$  __$$\ $$  __$$\ $$  __$$\ $$ |$$  __$$\ $$ |$$  __$$\ $$  _$$  _$$\ $$  __$$\\_$$  _|  $$  __$$\ $$ |  $$ |
$$ |  $$ |$$ /  $$ |$$$$$$$$ |$$ |  $$ |$$ |$$$$$$$$ |$$ |$$$$$$$$ |$$ / $$ / $$ |$$$$$$$$ | $$ |    $$ |  \__|$$ |  $$ |
$$ |  $$ |$$ |  $$ |$$   ____|$$ |  $$ |$$ |$$   ____|$$ |$$   ____|$$ | $$ | $$ |$$   ____| $$ |$$\ $$ |      $$ |  $$ |
 $$$$$$  |$$$$$$$  |\$$$$$$$\ $$ |  $$ |$$ |\$$$$$$$\ $$ |\$$$$$$$\ $$ | $$ | $$ |\$$$$$$$\  \$$$$  |$$ |      \$$$$$$$ |
 \______/ $$  ____/  \_______|\__|  \__|\__| \_______|\__| \_______|\__| \__| \__| \_______|  \____/ \__|       \____$$ |
          $$ |                                                                                                 $$\   $$ |
          $$ |                                                                                                 \$$$$$$  |
          \__|                                                                                                  \______/ 

Copy link
Member

Choose a reason for hiding this comment

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

The beauty of the install that is the opentelemetry-demo chart

=======================================================================================


 ██████╗ ████████╗███████╗██╗         ██████╗ ███████╗███╗   ███╗ ██████╗ 
██╔═══██╗╚══██╔══╝██╔════╝██║         ██╔══██╗██╔════╝████╗ ████║██╔═══██╗
██║   ██║   ██║   █████╗  ██║         ██║  ██║█████╗  ██╔████╔██║██║   ██║
██║   ██║   ██║   ██╔══╝  ██║         ██║  ██║██╔══╝  ██║╚██╔╝██║██║   ██║
╚██████╔╝   ██║   ███████╗███████╗    ██████╔╝███████╗██║ ╚═╝ ██║╚██████╔╝
 ╚═════╝    ╚═╝   ╚══════╝╚══════╝    ╚═════╝ ╚══════╝╚═╝     ╚═╝ ╚═════╝ 
                                                                          

@codecov
Copy link

codecov bot commented Aug 5, 2022

Codecov Report

Merging #3557 (6e1cc4d) into main (ce99d44) will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3557      +/-   ##
==========================================
+ Coverage   87.12%   87.17%   +0.05%     
==========================================
  Files         275      275              
  Lines        9959     9959              
==========================================
+ Hits         8677     8682       +5     
+ Misses       1282     1277       -5     
Impacted Files Coverage Δ
src/OpenTelemetry.Api/BaseProvider.cs 66.66% <0.00%> (-33.34%) ⬇️
...Listener/Internal/PrometheusExporterEventSource.cs 16.66% <0.00%> (-11.12%) ⬇️
...emetry.Api/Internal/OpenTelemetryApiEventSource.cs 73.52% <0.00%> (-8.83%) ⬇️
....Prometheus.HttpListener/PrometheusHttpListener.cs 78.66% <0.00%> (-4.00%) ⬇️
...ter.ZPages/Implementation/ZPagesActivityTracker.cs 97.14% <0.00%> (-2.86%) ⬇️
...nTelemetry/Internal/OpenTelemetrySdkEventSource.cs 79.16% <0.00%> (+1.66%) ⬆️
src/OpenTelemetry/Logs/OpenTelemetryLogger.cs 88.88% <0.00%> (+2.22%) ⬆️
...lemetry/Internal/SelfDiagnosticsConfigRefresher.cs 92.30% <0.00%> (+5.76%) ⬆️
...entation/ExportClient/OtlpGrpcTraceExportClient.cs 50.00% <0.00%> (+14.28%) ⬆️
...xporter.OpenTelemetryProtocol/OtlpTraceExporter.cs 59.09% <0.00%> (+22.72%) ⬆️

@utpilla utpilla merged commit 72f4e07 into open-telemetry:main Aug 5, 2022
@reyang reyang deleted the reyang/sanity branch August 5, 2022 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra Infra work - CI/CD, code coverage, linters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants