Skip to content
This repository was archived by the owner on Dec 26, 2024. It is now read-only.

XML guide added (Issue #461)#607

Draft
M4XPRD wants to merge 4 commits into
Hexlet:mainfrom
M4XPRD:add-guide-xml-461
Draft

XML guide added (Issue #461)#607
M4XPRD wants to merge 4 commits into
Hexlet:mainfrom
M4XPRD:add-guide-xml-461

Conversation

@M4XPRD

@M4XPRD M4XPRD commented Jun 7, 2023

Copy link
Copy Markdown
Contributor

Добавил гайд по XML

Ссылка на issue - #461

@vercel

vercel Bot commented Jun 7, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
hexletguides ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 18, 2023 9:56pm

@ssssank

ssssank commented Jun 26, 2023

Copy link
Copy Markdown
Contributor

Привет. Извини, что так долго, но с гайдами всегда так, пока их пишут нечасто

Пока гайд надо доработать, добавить в него больше пользы для читающих:

  • можно добавить что-нибудь про XSD и формирование моделей по схемам
  • лучше убрать пример про пространства имен именно в связке с html, так как он не очень явный, лучше придумать какой-нибудь другой пример
  • сравнения в конце с другими форматами, то же не очень нужны, так как гайд именно про xml
  • XML — дополнение к HTML - ты уверен в этом?

Посмотри вот эту статью, может быть почерпнёшь что-нибудь интересное: https://habr.com/ru/articles/524288/

@ssssank

ssssank commented Jul 4, 2023

Copy link
Copy Markdown
Contributor

@M4XPRD возьмёшься за доработку гайда? :)

@ssssank ssssank marked this pull request as draft July 10, 2023 08:03
@M4XPRD

M4XPRD commented Jul 11, 2023

Copy link
Copy Markdown
Contributor Author

Да, допилю. Ишью дополню (:

@M4XPRD

M4XPRD commented Jul 18, 2023

Copy link
Copy Markdown
Contributor Author

Дополнил и изменил гайд. Жду ревью 👓

@Hexlet Hexlet deleted a comment from vercel Bot Jul 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants