Nam, this looks better. At least links are working, and the website
visually looks similar and generally in good shape. I think there are still
issues. For example, I do not see any of the images (e.g. the beam logo on
top left is missing.)

On Wed, Apr 29, 2020 at 3:11 PM Brian Hulette <bhule...@google.com> wrote:

> I left a comment on the PR [1]. I think the reason all of the website
> content is not being tracked as file renames is because there was a series
> of commits that created files in the new directory, and then one commit
> that deleted the old directory. If there were a single commit with all of
> the deleted and new files, git would surely recognize they are effectively
> renameds and mark them as such. Maybe we just need to get all these commits
> squashed into one?
>
> [1] https://github.com/apache/beam/pull/11554#issuecomment-621489844
>

Nam, could you try this? If we can get git to recognize these as renames,
review process would be much easier.


>
> On Wed, Apr 29, 2020 at 10:39 AM Nam Bui <nam....@polidea.com> wrote:
>
>> Hi guys,
>>
>> I'm Nam - from the responsible team of Apache Beam website migration. I
>> am pleased to answer some of the questions here.
>>
>> @aizhamal: Thanks for informing to the community. :)
>> @altay, @robertwb: Yes. there is a problem with the staged version at the
>> moment. We didn't expect some behaviours on the build process. So, we fixed
>> it today and been waiting for @pablo to re-run it again. The purpose of
>> this PR is to migrate completely Beam site from Jekyll to Hugo. Therefore,
>> a bunch of deleted markdown files are from Jekyll which was located at
>> `beam/website/src`, and Hugo is located at `beam/website/www` now. In
>> `beam/website/README.md`, I wrote down about running the Hugo website
>> locally, although it is actually same as Jekyll (because it's also set
>> up with Docker & Gradle). In `beam/website/CONTRIBUTE.md`, I guided people
>> on how to get started with Hugo on the Beam website. There is also a link
>> in the "Translation Guide" section which points to a branch of multilingual
>> provenance, and it will become a next PR soon.
>>
>> Please let me know if you need more details. Feel free to ask any
>> questions and I will get back to you with answers. I'm so sorry if I answer
>> a little bit due to the timezone. :)
>>
>> Best regards,
>> Nam
>>
>>
>>
>> On Tue, Apr 28, 2020 at 8:49 PM Aizhamal Nurmamat kyzy <
>> aizha...@apache.org> wrote:
>>
>>> Adding +Nam Bui <nam....@polidea.com> and +Karolina Rosół
>>> <karolina.ro...@polidea.com> to follow up on questions.
>>>
>>> On Tue, Apr 28, 2020 at 11:34 AM Ahmet Altay <al...@google.com> wrote:
>>>
>>>> I am having trouble reviewing the staged version. What is the best way
>>>> to review this change?
>>>>
>>>> Do we expect any changes to markdown files, beyond some metadata?
>>>>
>>>> On Tue, Apr 28, 2020 at 10:45 AM Robert Bradshaw <rober...@google.com>
>>>> wrote:
>>>>
>>>>> Thanks. It'll be great to better support more languages.
>>>>>
>>>>> I looked at the PR and there seems to be no provenance/history. E.g.
>>>>> all the content seems to be entirely new files rather than diffs from the
>>>>> old. (There also seems to be a huge amount of auto-generated js code as
>>>>> well.)
>>>>>
>>>>
>>>> I agree. This makes it very hard to review. I also see a bunch of
>>>> deleted markdown files. Are they not getting migrated?
>>>>
>>>>
>>>>>
>>>>> On Tue, Apr 28, 2020 at 10:23 AM Aizhamal Nurmamat kyzy <
>>>>> aizha...@apache.org> wrote:
>>>>>
>>>>>> Hello everybody,
>>>>>>
>>>>>> We are almost done migrating the Apache Beam website from Jekyll to
>>>>>> Hugo. You can see the PR in [1], and we'd love to hear your
>>>>>> feedback/comments on the PR. It includes  detailed guidelines on
>>>>>> contributing to the new Hugo-based website and adding translations to 
>>>>>> pages
>>>>>> [2]. For those who are curious about adding new languages, we will 
>>>>>> provide
>>>>>> a proof of concept in the next couple of days in this thread.
>>>>>>
>>>>>> Since we want to move forward with the PR, I would like to ask the
>>>>>> community to hold off changes to the current Beam website for a week, 
>>>>>> until
>>>>>> we are able to review and merge the PR. Is this acceptable to everyone?
>>>>>>
>>>>>> In case anyone missed my previous email with the background for the
>>>>>> website migration, you can find more context here [3].
>>>>>>
>>>>>> Thanks,
>>>>>> Aizhamal
>>>>>>
>>>>>> [1] https://github.com/apache/beam/pull/11554
>>>>>> [2]
>>>>>> https://github.com/apache/beam/blob/256b7042bf504b94f161ca03b388a2ba247918d9/website/CONTRIBUTE.md
>>>>>> [3]
>>>>>> https://lists.apache.org/thread.html/r7fa6d710c0a1959cce5108e460d71c306ce5756cf96af818b41cb7ca%40%3Cdev.beam.apache.org%3E
>>>>>>
>>>>>

Reply via email to