Skip to content

master#5

Open
Ilyshnya wants to merge 6 commits intotypeundefined:masterfrom
Ilyshnya:master
Open

master#5
Ilyshnya wants to merge 6 commits intotypeundefined:masterfrom
Ilyshnya:master

Conversation

@Ilyshnya
Copy link
Copy Markdown

сделал дз

<script type="text/javascript" src="/scripts/jquery-3.4.1.js">
<meta charset="utf-8">
<script type="text/javascript" src="/scripts/jquery-3.4.1.js"></script>
<link rel="stylesheet" href="/scripts/bootstrap-4.0.0-dist/css/bootstrap.min.css">
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

А вы сами эти файлы не забыли вкомитить?

И, кстати, css отдавать по /scripts/* - это немного странно. Лучше завести еще один путь, например, /css/.

Copy link
Copy Markdown
Author

@Ilyshnya Ilyshnya Oct 25, 2019

Choose a reason for hiding this comment

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

я испавил,но не комитится почему-то,т.е. раньше видел коммит на странице,а сейчас нет,хотя пишет что коммит успешен,это из за того ,что вы мердже сделали ?И видите ли вы эти коммиты?Надо еще раз форк делать?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Нет, мерж я не делал. И в веб-морде я коммитов, где добавляются файлы bootstrap-4.0.0-dist я не вижу.
Попробуйте сделать git push из своего репозитория. Может быть просто коммит есть локальный, но вы его на сервер не запушили

Новый форк делать не надо, все и так сработает

Copy link
Copy Markdown
Author

@Ilyshnya Ilyshnya Oct 25, 2019

Choose a reason for hiding this comment

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

Сделал,вроде работает,спасибо.Теперь надо новы пул реквест делать?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Или все в этом комититься?,а то еще один пул реквест не получается сделать

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Нет, можно и нужно в этот

Copy link
Copy Markdown
Owner

@typeundefined typeundefined left a comment

Choose a reason for hiding this comment

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

Не вкоммичены файлы с bootstrap

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.

2 participants