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

Add serial monitor send/receive encoding options #7

Closed
wants to merge 1 commit into from

Conversation

AhmedSalihCezayir
Copy link
Owner

  • Add "send as " dropdown menu.
  • Add "receive as " dropdown menu.

Sending and receiving can be done in any encoding specified in StandardCharsets with the additional option to send comma-separated bytes and receive newline-separated bytes directly.

This fixes arduino/arduino-ide#1728 and offers an easy implementation for issue arduino/arduino-ide#1727.

@chid-bot
Copy link

chid-bot bot commented Jun 1, 2023

IMPACT GRAPH

Impact Graph

🟡 MODIFIED 🔴 REMOVED 🔵 POSSIBLY AFFECTED

📝 GENERAL INFORMATION

  • Number of Changed Code Files in This Pull Request 👉 4
  • Total Added Lines 👉 284
  • Total Deleted Lines 👉 187

📂 METRICS

Metric Category Numeric Value
Highly Buggy File Ratio 🟧 D 50.00%
Highly Code Churned File Ratio 🟨 C 50.00%
Pr Size 🟧 D 471
Impact Size 🟦 B 0.21%
RISK* 🟨 C 54.38%
*Risk is calculated as weighted sum of above metrics
Category Scala => Best: 🟩 A | Good: 🟦 B | Mediocre: 🟨 C | Bad: 🟧 D | Worst: 🟥 E

ℹ️ 💁 AUTHOR INFORMATION

  • The current status of the Ahmed Salih Cezayir is:
  • Current Pull Request Count 👉 3
  • Current Merged Pull Request Count 👉 0
  • Author Merge Rate Result 👉 0.00%

🌞 🌚 CO-CHANGE FILES INFORMATION

Missing Sibling Files of SerialMonitor.java Co Changed Ratio Co-Changed Count PR Count of SerialMonitor.java
app/src/processing/app/AbstractMonitor.java 50.00% 3 6
app/src/processing/app/Editor.java 50.00% 3 6
app/src/processing/app/NetworkMonitor.java 50.00% 3 6

🏆 QUALITY GATE

ALERT!

CONDITIONS

Risk score should be lower than 40%
Highly buggy file ratio should be lower than 20%
Highly churn file ratio should be lower than 25%
Pull request size should be lower than 400 lines of code

❗ POSSIBLE RISKY FILES

File Name Bug Frequency
SerialMonitor.java 50%
Serial.java 75%
File Name Current Code Churn
SerialMonitor.java 7.30
Serial.java 5.26

🔗 LINK OF THE WEBSITE

Click for Website

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.

Serial monitor character encoding option
1 participant