search

microsoft / pxt-ev3

Microsoft MakeCode editor for the LEGO MINDSTORMS EV3 Brick
https://makecode.mindstorms.com/
Other
55 stars 39 forks source link

Fix upload #1031

Closed dmitryaleshin closed 1 year ago

dmitryaleshin commented 1 year ago

1 Add baudRate property and assign it with existing baudrate on connect 2 Read full packet from brick instead random one

THEb0nny commented 1 year ago

вы это проверяли? baudRate я тоже исправлял. did you check it?

dmitryaleshin commented 1 year ago

вы это проверяли? baudRate я тоже исправлял. did you check it?

Да я проверял. В комите два изменения. Дело не только в кейсе имени. Русский мне комфортен. Sure i did. It works for me. The fix is not only in property case but the brick communication as well.

THEb0nny commented 1 year ago

Да, действительно работает, я проверил, но надо у вас там код немного подкорректировать... Вы красавчик!)

THEb0nny commented 1 year ago

Блин, я не очень понимаю как на github код этого pull request подкорректировать.

dmitryaleshin commented 1 year ago

Блин, я не очень понимаю как на github код этого pull request подкорректировать.

Думаю никак. Цитирование кода и комментарии. Суть PR, видимо, в этом image

THEb0nny commented 1 year ago

Блин, я не очень понимаю как на github код этого pull request подкорректировать.

Думаю никак. Цитирование кода и комментарии. Суть PR, видимо, в этом image

Комментировать я вижу как, а вот разрабы могут прям делать свой вариант и остаётся только его принять или отклонить...

THEb0nny commented 1 year ago

Может можно я создам свой pull request с сылкой на ваше изменение? А вы закроете тут pull request.

dmitryaleshin commented 1 year ago

Может можно я создам свой pull request с сылкой на ваше изменение? А вы закроете тут pull request.

Да мне без разницы. Я показал, как можно поправить проблему, на которую забили, вот и все.

THEb0nny commented 1 year ago

Может можно я создам свой pull request с сылкой на ваше изменение? А вы закроете тут pull request.

Да мне без разницы. Я показал, как можно поправить проблему, на которую забили, вот и все.

Хорошо, давайте так как я описал.

THEb0nny commented 1 year ago

@dmitryaleshin можно закрыть этот pull request.

THEb0nny commented 1 year ago

@dmitryaleshin изменения включены, версия создана, закройте этот pull request.