Skip to content

Commit d550256

Browse files
authored
Delete the website and references (carbon-language#782)
.pre-commit-config.yaml and proposals/p0540.md (linking goals.html, fixed to goals.md link) are the two references I could find. The site is disabled, though; I think it makes the most sense to delete supporting code, and in the future hopefully we can use GH pages to publish.
1 parent 4cbf5c6 commit d550256

File tree

7 files changed

+7
-32
lines changed

7 files changed

+7
-32
lines changed

.pre-commit-config.yaml

+1-8
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,7 @@ repos:
1919
- id: check-executables-have-shebangs
2020
- id: check-merge-conflict
2121
- id: check-symlinks
22-
exclude: |
23-
(?x)^(
24-
website/jekyll/site/_includes|
25-
bazel-(clang-toolchain|execroot)
26-
)$
22+
exclude: ^bazel-(clang-toolchain|execroot)$
2723
- id: check-yaml
2824
- id: detect-private-key
2925
- id: end-of-file-fixer
@@ -71,7 +67,6 @@ repos:
7167
.bazelversion|
7268
compile_flags.txt|
7369
third_party/examples/.*/compile_flags.carbon.txt|
74-
website/(firebase/.firebaserc|jekyll/(Gemfile.lock|theme/.*))|
7570
.*\.def|
7671
.*\.svg|
7772
.*/testdata/.*\.golden
@@ -83,7 +78,6 @@ repos:
8378
hooks:
8479
- id: codespell
8580
args: ['-I', '.codespell_ignore', '--uri-ignore-words-list', '*']
86-
exclude: '^website/jekyll/Gemfile.lock$'
8781
- repo: local
8882
hooks:
8983
- id: executable-semantics-tests
@@ -116,7 +110,6 @@ repos:
116110
rev: bc48c541add1551be726f23c4294c773442341cb # frozen: v2.3.2
117111
hooks:
118112
- id: prettier
119-
exclude: ^website/jekyll/theme/
120113
# Run linters last, as formatters and other checks may fix issues.
121114
- repo: https://gitlab.com/PyCQA/flake8
122115
rev: dcd740bc0ebaf2b3d43e59a0060d157c97de13f3 # frozen: 3.9.2

BUILD

+1-5
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,4 @@
22
# Exceptions. See /LICENSE for license information.
33
# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
44

5-
filegroup(
6-
name = "website_content",
7-
srcs = glob(["*"]),
8-
visibility = ["//website/jekyll:__pkg__"],
9-
)
5+
# Empty stub.

docs/BUILD

-9
This file was deleted.

proposals/BUILD

+2-5
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,7 @@
33
# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
44

55
filegroup(
6-
name = "website_content",
6+
name = "md_files",
77
srcs = glob(["*.md"]),
8-
visibility = [
9-
"//proposals/scripts:__pkg__",
10-
"//website/jekyll:__pkg__",
11-
],
8+
visibility = ["//proposals/scripts:__pkg__"],
129
)

proposals/p0540.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ Remove `Void` from the Carbon design.
3939
## Rationale based on Carbon's goals
4040

4141
Eliminating `Void` will make Carbon code
42-
[easier to read, understand, and write](https://carbon-lang.dev/docs/project/goals.html#code-that-is-easy-to-read-understand-and-write).
42+
[easier to read, understand, and write](/docs/project/goals.md#code-that-is-easy-to-read-understand-and-write).
4343
The main advantage of `Void` is that it is recognizable and familiar to C++
4444
programmers. However, we haven't yet found any use cases where using `Void`
4545
results in clearer code, even to programmers transitioning from C++. In

proposals/scripts/BUILD

+2-3
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,12 @@ load("@rules_python//python:defs.bzl", "py_binary", "py_library", "py_test")
77
py_library(
88
name = "proposals",
99
srcs = ["proposals.py"],
10-
visibility = ["//website/jekyll:__pkg__"],
1110
)
1211

1312
py_test(
1413
name = "proposals_test",
1514
srcs = ["proposals_test.py"],
16-
data = ["//proposals:website_content"],
15+
data = ["//proposals:md_files"],
1716
python_version = "PY3",
1817
deps = [":proposals"],
1918
)
@@ -27,7 +26,7 @@ py_binary(
2726
py_test(
2827
name = "new_proposal_test",
2928
srcs = ["new_proposal_test.py"],
30-
data = ["//proposals:website_content"],
29+
data = ["//proposals:md_files"],
3130
python_version = "PY3",
3231
deps = [":new_proposal"],
3332
)

setup.cfg

-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
[flake8]
66
max-line-length = 80
7-
exclude = website/jekyll/build
87
# E203: This warning is not PEP 8 compliant.
98
# E402: Allow the pythonpath modifications before repo-local imports.
109
# W503: flake8 v3.8.4 is inconsistent with black v20.8b1 (pre-commit run -a).

0 commit comments

Comments
 (0)